-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Unaccent layers, bookmarks, actions, layouts and settings in locator and processing toolbox search #64436
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
always perform unaccent to both the layer name and search term. This allows users to find layers regardless of accents - for example, searching for "creme" will match a layer named "crème", and vice versa.
🪟 Windows Qt6 buildsDownload Windows Qt6 builds of this PR for testing. 🍎 MacOS Qt6 buildsDownload MacOS Qt6 builds of this PR for testing. |
Makes the Settings and Actions locator filters always perform accent-insensitive matching by applying QgsStringUtils::unaccent() to both the item name and search term before scoring. E.g. searching for "parametres" will match "Paramètres généraux".
|
Does it fix #39475 ? |
Partially, as it does fix the unaccent for the locator search bar. I haven't looked at the processing search yet. |
|
@nicogodet It does fix it now. |
a8f8fde to
e64f0d0
Compare
fixes qgis#39475 This resolves the issue where searching for "selection" would not match algorithms named "Sélection par localisation" and similar cases with accented characters.
Nice! |
e64f0d0 to
cc72b2e
Compare
|
Thanks @lbartoletti |
Use the unaccent function to improve locator and search more easily for layer names, layouts and bookmarks.
Unaccent also now works for the search in the processing toolbox.
E.g. from now on if a user searches for "creme" the locator will match a layer named "crème", and vice versa.
I tested all by hand (except the locator actions filter as I got lazy). Sometimes the CI fails but unrelated. I don't know if the processing toolbox search should have come in another pull request, I just added it here.
Fixes #39475