Community
Participate
Working Groups
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. In the "Content" tab the TOC order can be configured (via HELP_DATA), in contrast in the "Search Results" tab the result categories are always alphabetically ordered. This feature request is about modeling search result categories ordering after TOC ordering because this is what the user expects. Example: Searching for "What's New" -> Search Results tab: <http://help.eclipse.org/indigo/advanced/searchView.jsp?searchWord=What%27s%20New&showSearchCategories=true&scope=All%20topics> "Workbench User Guide" containing "Eclipse Platform What's New in 3.7" should be at the top (like in the "Content" tab) but is somewhere at the middle/bottom.
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.