-
-
Notifications
You must be signed in to change notification settings - Fork 446
Remove Subtitle Highlight #890
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
Conversation
This reverts commit 1a49178.
which plugin has highlighting on subtitle? also we need to remove the code too as this is only removal of the view |
I looked around and there is no plug-in that uses this part specifically.
This part operates in the highlight converter. Compared to the query, the code that puts colors in is used in duplicate in two places - title and subtitle. In other words, only the subtitle is not separately worked. I think we only need to erase the view code. |
Searching WindowSettings subtitle might be useful if you want to find all settings related to e.g. Privacy
@onesounds @taooceros well that's the thing, do we want to completely rip out the subtitle highlighting or just 'disable' it and keep the code behind intact? |
I didn't think it was a function to "use" highlights. Highlight is just to indicate the matched letters. I only considered the possibility of some different kind of function in this part. For example, in the case of a home assistant plug-in, the highlights operate differently depending on on on/off. Of course, this plug-in doesn't work like that.
The same applies to other plug-ins. Although not in name, there may be cases where it matches in path. This function is a function that tells you which part matches the user's search intention when searching. However, in general, it often appears overlapping in the path area, which damages the visual design. Also, erasing this function does not damage the actual path search results, This is ultimately the same as Title Highlight, but considering the user's general search intention, it is correct that highlights exist only in Title. If we intended to search for Subtitle, we wouldn't question the search results even if we didn't show the highlights Also, even if it was not intended to be so, there is no reason to question the current search results. This is because the search accuracy is high. This function has a reason to exist if there are really often unintended search results, but in my opinion, it does not exist.
Your intention is understood by me as 'delete unused code'. However, this highlight code is just a code that checks and highlights whether the designated text block matches the query. How can we take out the subtitle highlight? We should use Title Highlight. Is it possible to separate only the subtle highlight? |
yeah that's what i mean, take out the code that offers the subtitle highlighting (they are also found in the api so we need to deprecate the method as well).
sorry for the confusion, i meant, subtitle search will be kept but remove subtitle highlighting, and yes i agree, search accuracy is high enough you wont need to search the subtitle most of the time. Just so we are on the same page, when user search something like this, it should be fine that 'computer' in the subtitle is no longer highlighted: |
OK. I didn't know there was one in API. If so, it would be better to remove the code from the relevant plug-in, and if so, it is not simple. It's not urgent, so let's proceed slowly. |
No, easy change. I'm just clarifying that this change will mean if plugin offers searching subtitle and when matched like the screenshot above, the subtitle matched will no longer be highlighted. If this is better design wise as you have already indicated, let's proceed. |
Is there no UI fix to this issue? Maybe a more subtle highlight in the subtitle (say, bold instead of blue) ? Removing this entirely seems like a drastic measure...
The point is to also see why a result matched the typed query...
Disagree, but I might be biased since I added this a long, long time ago (5+ years, time flies...) |
Hi, I know you are an
There is no setting. I think it is unnecessary.
Why should the user know which part was matched for typed query? If this is really necessary, we'd rather remove the highlights in the all area, and just show the score.
|
I know that, too 😢
I mention this as an example, because I remember it as the main reason people requested this feature back in the day.
You assume that plugins are searching things, which is not always the case... The calculator plugin is not searching things, the Spotify plugin is not always searching things, and so on... There has to be some creative/practical use of highlighting the subtitle other than to show a search match... Why completely kill that possibility?
This project is supposed to target "power users", and modern design is very much not afaik 😛
All I'm arguing for here is: why not just go with the original plan? add a toggle, make it off by default...? |
At that time, I don't know why users requested it. However, it may have been necessary in that the results were not appropriate. However, the search results are excellent now. I don't think users have come up with the right solution.
Currently, there is no plug-in using such a function. There is a possibility, but if it is a plug-in with such a possibility, I think it is better to use it more in other factors. In places such as icons and title areas. Giving a degree of freedom in this area undermines design consistency.
I saw the code, and it has such a difficult structure to apply. We need to make a new highlight converter. I concluded that the value for the cost was low. If you make it, it doesn't matter if this function goes into setting. how about that? |
I would say design is still very much important whilst this project targets power users, it would also be a win if we could get beginner to intermediate users on board too. I always was under the impression that the highlighting was only implemented to help users see how a query is matched. @JohnTheGr8 i suppose the functionality could have further usage, we can always reimplemented specifically for that potential future usage, which will be better designed than to re-use the existing structure. Worst case we can just bring back the code again from the commit history. |
#68
I first pr considered on/off setting, but I changed my mind. It's right to just erase this.
Tested: