| Summary: | [Help][Search] Model result categories ordering after TOC ordering | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Holger Voormann <eclipse> | ||||
| Component: | User Assistance | Assignee: | Holger Voormann <eclipse> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | enhancement | ||||||
| Priority: | P3 | CC: | Lars.Vogel | ||||
| Version: | 3.7.1 | Keywords: | greatfix | ||||
| Target Milestone: | 4.5 M6 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: |
https://git.eclipse.org/r/41365 https://git.eclipse.org/r/21670 https://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=4564d3cceabcf7155dd1ceb4da87f5c65bb77578 |
||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Holger Voormann
Created attachment 207322 [details]
Proposed patch: use ProductPreferences.getTocOrder() to sort search result categories
Any chance to get this in 3.8/4.2? Anything more I should supply? Gerrit patch set: https://git.eclipse.org/r/21670 Problem: By default the search result categories are on a par with the top-level books (TOCs), but the order of the TOCs and the order of the result categories can differ. For example, if you search for "subsystem" (http://eclip.se/1r) and "Show result categories" is enabled, then the category "Workbench User Guide" is shown in the search result list at the bottom, in contrast to the table of contents where the "Workbench User Guide" book is shown at the top. From the user perspective this is confusing. Caused by: Both the books in the Contents tab and the categories in the Search Results tab are by default alphabetically ordered. The table of content book order can be changed via the Help Data file (see http://eclip.se/1s), for instance in the Eclipse IDE the "Workbench User Guide" is shown at the top. In contrast, the search results category order cannot be changed. Solution: The search results category order should follow the table of content book order. Thanks Holger for the Gerrit review, looks like I should try to start monitoring ua also for patches (not sure if I will find the time). I try to review during M6. New Gerrit change created: https://git.eclipse.org/r/41365 Gerrit change https://git.eclipse.org/r/21670 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=4564d3cceabcf7155dd1ceb4da87f5c65bb77578 Sorry Holger for the long delay in review, eclipse.ua is unfortunately not my area of expertise. But this valuable contribution was long enough in our queue. It looks good to me, so I merge it so that more people can test it. AFAIK we do not have an expert in help anymore active in the team so I commit this hoping not to break anything. Thanks a bunch for the contribution and sorry again for the delay in the review. |