Deleted bar, progress and progressbar directives

[lizozom]

Deleted directives

• progress
• bar
• progressbar

Changed

• implementation of Discover > string field > proressbar to use EUI progressbar and tooltip.

The new implementation:

The old implemtation:

The big difference is that the EUI progress bar doesn't support showing the absolute value on the bar, so I have placed it next to the progress bar instead.

Summary

Summarize your PR. If it involves visual changes include a screenshot or gif.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

• This was checked for cross-browser compatibility, including a check against IE11
  - [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)
  - [ ] Documentation was added for features that require explanation or tutorials
  - [ ] Unit or functional tests were updated or added to match the most common scenarios
• This was checked for keyboard-only and screenreader accessibility

For maintainers

• This was checked for breaking API changes and was labeled appropriately
• This includes a feature addition or change that requires a release note and was labeled appropriately

[elasticmachine]

Pinging @elastic/kibana-app

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[lizozom]

elastic/eui#1749

[ryankeairns]

@lizozom This is looking nice! One thing that seems to be missing between the old and new is that the length of the progress bar (background) conveyed additional information - presumably, it is the % each individual category makes up of the entire record set? Perhaps that was intentionally removed, please disregard if that is the case.

Separately, I think Dave recently made a change where the filtering icons (magnifying glass +/-) only display on hover since they repeat visually. He made this change in the list/table portion of the Discover page (I think), and it would be nice to do the same, here, although this could also be done in a separate PR.

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[lizozom]

@ryankeairns Thanks for reviewing this!
The length of the progressbar actually didn't convey additional information.
I found that confusing as well. The total length of the progress bar was defined by the percentage, and the "white part" is actually just created by the number. The absolute amount of items in each bucket is shown using a tooltip in both cases.

I also found the "magnifying lens" icons current design unappealing, but I think that this should be done in a separate PR.

[ryankeairns]

@lizozom aha! In that case, this new design definitely seems more accurate :)

We (the design team) can get those icons cleaned up. It's a little busy over there, but there is a pattern on how to improve it. Thanks!

[lizozom]

@ryankeairns if so would you mind approving?

[ryankeairns]

LGTM

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[elasticmachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request

[elasticmachine]

Pinging @elastic/eui-design (EUI)