| Summary: | Trim the white-space before a search mach output to improve match visibility | ||
|---|---|---|---|
| Product: | [Applications] kate | Reporter: | Bill in KCMO <billthetailor> |
| Component: | search | Assignee: | KWrite Developers <kwrite-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | kare.sars, winter |
| Priority: | NOR | ||
| Version First Reported In: | 23.08.5 | ||
| Target Milestone: | --- | ||
| Platform: | Kubuntu | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/utilities/kate/-/commit/7b3e5ea11a382bc84df9f1070413f432e0918db0 | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: |
Including indents in search results pushes the results too far to the right.
Better to omit indentation so that all results can be seen |
||
|
Description
Bill in KCMO
2025-09-03 03:23:28 UTC
Created attachment 184656 [details]
Better to omit indentation so that all results can be seen
Thanks for the report. I turned this into a wish. It is a good improvement suggestion! Sometimes the indentation can have some value, so a blanket remove all leading white-space might not be desired. The target should definitely be to get the match to always be visible. This needs some tweaking. as a follow-on to this, someone just asked about tab widths in the search results. https://discuss.kde.org/t/size-of-tab-in-katesearch-and-diff-plugin/40155 not quite the same thing but related in the sense that whitespace handling in the search results might be improved. A possibly relevant merge request was started @ https://invent.kde.org/utilities/kate/-/merge_requests/1903 A possibly relevant merge request was started @ https://invent.kde.org/utilities/kate/-/merge_requests/1906 Git commit 7b3e5ea11a382bc84df9f1070413f432e0918db0 by Christoph Cullmann, on behalf of Kåre Särs. Committed on 04/10/2025 at 19:19. Pushed by cullmann into branch 'master'. S&R: Limit the pre-match part in the results to 25% of view width If the search tool-view is narrow we don't want too much pre-match string as the match might not even be visible. So ensure that at most 25% of the tree-view width is occupied by the pre-match part M +13 -4 addons/search/SearchResultsDelegate.cpp https://invent.kde.org/utilities/kate/-/commit/7b3e5ea11a382bc84df9f1070413f432e0918db0 |