Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Discover] Update Include/Exclude Filter Icons #146802

Open
MichaelMarcialis opened this issue Dec 1, 2022 · 3 comments
Open

[Discover] Update Include/Exclude Filter Icons #146802

MichaelMarcialis opened this issue Dec 1, 2022 · 3 comments
Assignees
Labels
blocked Feature:Discover Discover Application impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL.

Comments

@MichaelMarcialis
Copy link
Contributor

New icons have recently been created for include/exclude filter actions (filterInclude, filterExclude). The purpose for their creation was to more clearly differentiate filter action icons and reduce our current overuse of the plusInCircle/minusInCircle icons throughout Kibana interfaces.

In this particular issue, we'd like to update the Discover apps use of the plus/minus icons for filtering and replace them with these new include/exclude filter icons in the following locations (and any other relevant areas in the app I may have missed here):

  • Field popover filter actions
  • Table cell filter actions

image

Icon D @ 12px

CCing @ninoslavmiskovic, @andreadelrio.

@MichaelMarcialis MichaelMarcialis added Feature:Discover Discover Application Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. labels Dec 1, 2022
@elasticmachine
Copy link
Contributor

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

@andreadelrio andreadelrio self-assigned this Dec 1, 2022
@kertal kertal added the impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. label Dec 6, 2022
@davismcphee davismcphee added the loe:needs-research This issue requires some research before it can be worked on or estimated label Sep 8, 2023
@rdner
Copy link
Member

rdner commented Jun 11, 2024

Hey @MichaelMarcialis, is it still going to be addressed at some point? I see that the linked PR was closed, although new icons looked good to me.

I personally struggle with this problem every time I use Kibana Discover because the same "plus in the circle" icon is used for two unrelated actions:

Screenshot 2024-06-11 at 09 54 48 Screenshot 2024-06-11 at 09 54 58

My brain refuses to remember that the plus on the right picture is filtering, this is not obvious at all. I would expect any filter-related actions to have a common funnel-like icon like, not the same icon we use for adding a column to the table view.

At least on one occasion an accidental filter by value interfered with my investigation while looking at the logs and led to incorrect results.

@kertal kertal added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. and removed impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. labels Jun 11, 2024
@MichaelMarcialis
Copy link
Contributor Author

Thanks for the feedback, @rdner. We temporarily paused implementation of the new add/remove filter icons in Discover due to some concern over their size. That said, I don't think we plan to abandon the change; just adjust the icons to see if we can account for the aforementioned size concerns.

CCing @ninoslavmiskovic, @kertal, @andreadelrio. Is this something we care to boost in priority? If so, I can take another crack at addressing those concerns. I snuck the updated icons as part of my One Discover context aware mockups to remind folks they exist ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Feature:Discover Discover Application impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants