Manageiq-ui-classic: Semi-redundant check all checkboxes for C&U collection trees

[skateman]

Do we need these Check all checkboxes for the two trees? The switch above basically does the exact same thing. Removing the checkboxes would simplify code complexity by a lot.

[skateman]

@miq-bot add_label question, cleanup

[h-kataria]

@skateman let's get rid of "Check All" checkboxes since they are doing the same thing as the switch above them.

[h-kataria]

@skateman when i toggle the switch back to "No" it does not uncheck all the items in the tree, which is what "Check All" checkbox used to do, with the change in this PR user will have to manually check/uncheck the items if they want to check/uncheck them all.
Let's keep a note of this and if needed, we can add Check All checkbox back once trees are converted to React.

[Fryguy]

Just saw this, but yeah, Check All is more of a convenience for checking everything so you can then uncheck individual items (or vice versa, to uncheck all boxes so you can then check individual items).