-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Confusing terminology: Saved search and saved query #153809
Comments
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
@stratoula Correct me if I'm wrong, but I think the term "filter set" got thrown around as a replacement for "saved query" (although I'm not aware of current plans to actually rename it). |
Yes we haven't concluded on the naming yet, filter set was one suggestion, saved search was another. |
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
While I think renaming |
I agree but still both terminologies are wrong (saved search for discover view, saved query for unified search). |
Adding some other options:
|
I'm wondering if we could instead consider merging these two objects into a single concept.
the main difference is in the additional Discover configuration applied to the table, but the search bar and filters are the same. If we instead have a single concept of "search configuration" (don't think about the naming here) that is reusable from Discover and from the unified search? the "discover view" could be just an extension to that, that is saved together with the "search configuration" and can be used elsewhere, like today in Dashboard.
|
@markov00 I really like this idea and is possibly a smaller breaking change ( we can keep the saved-search for Discover and migrate the saved-queries) |
I like your idea / concept also @markov00 👍 the only comment I have is the same you also have namely the naming "Search configuration". The plus of using "Saved..." is that it indicates to the user that is something they store/save. I don't know the right answer, but perhaps the copy team @gchaps could come up with suggestions. @jughosta also proposed some good ones. |
How about |
I'm +1 for a single concept here as well. I'm not sure if there are reasons users currently would prefer to keep the selected fields/columns or not in discover (knowingly or not), but perhaps it's included as an option when saving? |
@mdefazio this is because you can open a "saved search" in Dashboard and it shows up as a table with the selected fields/column |
Until we have a single concept, I'd propose that we go with @markov00's suggestion and update the copy. One question though. Can we remove the I found the 'save saved` copy in 2 additional places: Following @markov00's suggestion, the copy then becomes: In the screenshot on the left:
In the screenshot on the right
Are there any other places that "saved" appears in this workflow? |
@gchaps I can remove the title so this will look like and if I load a query it will look like: I think it is fine |
Implementation of the aforementioned changes here #154517 |
@gchaps Do we need to provide different text when a query is loaded? Was this previously unclear for users what would happen when choosing this option? Some thoughts on the above screen... I understand if this is going outside the scope of the original issue. So I apologize. This just also struck me as a bit repetitive. |
@stratoula @gchaps what if we further simplify the menu like this? |
@gchaps and I worked through a collection of copy edits shown in this prototype: QueryMenu.mp4I like @andreadelrio 's idea above as it solves the odd interaction of clicking on the popover footer button, but showing a 2nd context menu. I think one possible edit could be that we don't need both Update: List of copy edits below: |
@andreadelrio @gchaps Mind taking another look at the summary of copy / minor UI edits to make sure we're all in agreement? Thank you! |
In my proposal, I'm also pushing all the saved queries related elements/text to the bottom of the popover. That way the popover is a Also, if we remove the add filter button (which already has an exposed dedicated button) then (1) would just be bulk filter actions. |
## Summary Introduces the changes that are mentioned here #153809 (comment) <img width="481" alt="image" src="https://user-images.githubusercontent.com/17003240/230306087-6e82bca8-a5c0-4a13-b7d1-b04fca8ac374.png"> <img width="460" alt="image" src="https://user-images.githubusercontent.com/17003240/230306169-ece13204-da76-495e-b258-4bfea0877118.png"> <img width="315" alt="image" src="https://user-images.githubusercontent.com/17003240/230306267-955161c4-a5cf-4427-beef-b768dea38c00.png"> --------- Co-authored-by: Kibana Machine <[email protected]>
## Summary Introduces the changes that are mentioned here elastic#153809 (comment) <img width="481" alt="image" src="https://user-images.githubusercontent.com/17003240/230306087-6e82bca8-a5c0-4a13-b7d1-b04fca8ac374.png"> <img width="460" alt="image" src="https://user-images.githubusercontent.com/17003240/230306169-ece13204-da76-495e-b258-4bfea0877118.png"> <img width="315" alt="image" src="https://user-images.githubusercontent.com/17003240/230306267-955161c4-a5cf-4427-beef-b768dea38c00.png"> --------- Co-authored-by: Kibana Machine <[email protected]>
I like @andreadelrio's proposal to push all saved query related elements to the bottom and removing the using the query name in the menu item. For the bulk filter actions, I think we still need to include the word "filter" somewhere in the copy. @mdefazio would this work with the designs that you proposed? |
If we're changing this terminology, does it make sense to update the blog post? |
My hesitation with removing the popover title is if there are no edits to the saved query, then the save option would be disabled and this would be the only way to see which query I have selected. I'm fine keeping Do we want to move some of these UI specific changes to another PR or Issue? The main issue here is the terminology. So whether we show @stratoula I know you've already done some work on these edits. Apologies for the updates after-the-fact, but do you have a preference on if it makes sense to split up this work? |
@mdefazio yes I would prefer to use this issue for terminology changes and move everything else in another ER. Can you create one and send it to me? Also if we have more terminology changes can someone mention them here in a comment in order to do them before the FF? Thanx a ton! |
@stratoula No problem. I've cut out the broader changes that we've discussed (reordering menu items and including saved searches) and only showing the copy/terminology changes. Ok...so there's a few minor UI edits, but I think these are important to be done alongside these copy edits. I'm not mentioning the icon changes here—I think those can be done in the followup ER (which I'm working on now).
I've kept this button in place for now so this only is removing a button and updating copy. But this will be updated again for the next release. Please advise if you or anyone wants to break this up differently
Let me know if you have any questions! Thank you. |
@mdefazio I love this redesign, thanks for that. I have a minor thing to propose, discuss. |
@mdefazio thanx, this is very helpful. I am not sure if this will make it for 8.8. Just a comment on the changes |
@stratoula Thank you for the heads up—makes sense and I suspect the terminology updates outside of this will help regardless. @markov00 I agree with your thinking. I will wrap up my enhancement request today that summarizes our other proposed changes. This is something absolutely worth including there. I'll be sure to tag you on it when I have it up. |
## Summary Makes the changes described on #153809 (comment) <img width="481" alt="image" src="https://user-images.githubusercontent.com/17003240/233070006-42d45555-a469-4aa7-969c-b121f9761b76.png"> <img width="435" alt="image" src="https://user-images.githubusercontent.com/17003240/233070100-9ee0ea02-ce4f-4ee0-904d-f55754d40b56.png"> ### Checklist - [x] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/packages/kbn-i18n/README.md) - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
Closing in favor of #155528 This changes described here #153809 (comment) have already been merged |
Related: #174144 |
Perhaps relevant for this group as well, a proposal on how context menus can be customized: elastic/eui#7507 |
Kibana uses the terms saved search and saved query, which are close in wording and easily confused:
Let's renew the discussions on renaming these two objects. For example:
We'll need to revisit the copy in these two screens:
The text was updated successfully, but these errors were encountered: