[UnifiedHistogram] Redesign Buttons, Breakdown and Interval

[kertal]

Tasks

• All buttons above the visualization should utilize the Shared UX toolbar button styles (again, to differentiate from other priority page actions).
• Change Breakdown should be migrated to a button selector rather than the current select element.
• Visualization legend should no longer use dots for color representations.
• Migrate Time Interval selection to button selector element
• Migrate "Hide visualization" of the options menu to it's own hide/show button displayed
• Remove options menu

[elasticmachine]

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

[davismcphee]

Since Visualization options menu removed, there's no more way restore default height to visualization, so it's no longer needed?

I think this was excluded to reduce clutter after it was determined to have very low usage in FullStory. @MichaelMarcialis should be able to confirm.

We need to migrate the number of hits to the data table, since this is currently implemented in the histogram code, and can be provided by the Lens visualization, this is a bigger refactoring. If we can't do this, the design looks a bit different in the first version, since we can't used the space where the number of hits are currently being displayed

@kertal I think you had suggested possibly doing similar to the EUI data grid render functions and pass the total hits component to Discover through Unified Histogram, which seemed like a good idea IMO. This would mean Unified Histogram could still control the logic, but we could disable the top hits display and render it wherever we want in the data grid panel.

[kertal]

@davismcphee

Since Visualization options menu removed, there's no more way restore default height to visualization, so it's no longer needed?

I think this was excluded to reduce clutter after it was determined to have very low usage in FullStory. @MichaelMarcialis should be able to confirm.

Now that we have also the fieldlist width scaleable, and also there is now way to reset it, we should align this behavior IMO

@kertal I think you had suggested possibly doing similar to the EUI data grid render functions and pass the total hits component to Discover through Unified Histogram, which seemed like a good idea IMO. This would mean Unified Histogram could still control the logic, but we could disable the top hits display and render it wherever we want in the data grid panel.

I had a brief look and we can get the number of hits in Discover by e.g. dataState.data$.totalHits$.getValue().result, so I think we are already able to apply this change without much refactoring.

[MichaelMarcialis]

Since Visualization options menu removed, there's no more way restore default height to visualization, so it's no longer needed?

I think this was excluded to reduce clutter after it was determined to have very low usage in FullStory. @MichaelMarcialis should be able to confirm.

Now that we have also the fieldlist width scaleable, and also there is now way to reset it, we should align this behavior IMO

@kertal, @davismcphee: Correct. When I last checked FullStory, the "Reset to default height" button was only used by 0.15% of unique users over the last 30 days. Considering that very low usage, the increase in discoverability we're proposing to the show/hide visualization button, and the fact that users can simply use the resize functionality at-will, I think we can confidently remove this feature to help reduce clutter.