Bug 85729 - [Help][Context] Element order returned by IContext.getRelatedTopics() should not be changed
Summary: [Help][Context] Element order returned by IContext.getRelatedTopics() should ...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M3   Edit
Assignee: platform-ua-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2005-02-17 11:58 EST by Dani Megert CLA
Modified: 2007-10-18 17:25 EDT (History)
1 user (show)

See Also:

Attachments
Patch for 3.4 stream (2.47 KB, patch)
2007-06-13 19:34 EDT, Adam Archer CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2005-02-17 11:58:03 EST 
I20050217-0800

I tried to put my items (grouped by category "Java help:") after "See also:"
this was not possible.

A client who decides to provide dynamic help knows best how the sequence should
be and hence this should not be overridden.
Comment 1 Dejan Glozic CLA 2005-02-17 12:12:57 EST 
That is fairly simple - we just comment out the line that sorts the links.
Comment 2 Adam Archer CLA 2007-06-13 19:34:24 EDT 
Created attachment 71248 [details]
Patch for 3.4 stream

If we simply commented out the sorter, then having multiple entries with the same category spread throughout the array would lead to duplicate headings.

This patch changes the sorter logic so that it will simply cluster items of the same category together, but will maintain the order in which categories are first discovered. So if your first entry uses the default (null) category, then "See Also:" will appear on top, if your first entry uses something else, then that category will appear on top.
Comment 3 Chris Goldthorpe CLA 2007-10-18 17:25:05 EDT 
Fixed in HEAD and added some JUnits.