Manageiq-ui-classic: Toolbar simplification

[epwinchell]

Currently, we display an icon for each toolbar group. This predates the implementation of Patternfly and contributes to the real estate problem experienced in the toolbars. For example: we have never had room in the toolbar to add filtering/search and have significant wrapping at some resolutions.

PF3 & PF4 examples use icons very sparingly:
https://www.patternfly.org/v3/pattern-library/forms-and-controls/toolbar/
https://www.patternfly.org/v4/documentation/core/components/toolbar

I would like to suggest that we remove all top level icons when text is also present, with the exception of custom button groups.

Before

After

Implementing: https://github.com/ManageIQ/manageiq-ui-classic/issues/5892 will also help a lot.

[epwinchell]

@Loicavenel @terezanovotna @martinpovolny

[terezanovotna]

This is a great idea @epwinchell. This is going to make this toolbar much more simple.

[terezanovotna]

@miq-bot add_label ux/approved

[martinpovolny]

Ok. Let's do this after or with https://github.com/ManageIQ/manageiq-ui-classic/pull/5997

[himdel]

First impression: this makes the toolbars confusing, suddenly every button is the same

I would not go so far as to claim we should only keep the icons, not the text,
but I do think the icons are what people distinguish the options by.
(The text is still useful the first few times, when searching, etc.)