By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1849 - [Help Handbooks] Help handbook table of contents sorting is confusing
Summary: [Help Handbooks] Help handbook table of contents sorting is confusing
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: 2014
  Show dependency treegraph
 
Reported: 2014-01-22 17:15 CST by Darrell
Modified: 2014-10-04 18:20 CDT (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2014-01-22 17:15:56 CST
Currently the sorting method used in the help handbook table of contents is ASCII. Upper case app names sort before lower case app names.

Sorting would be more helpful if not case sensitive.
Comment 1 Darrell 2014-02-08 18:26:51 CST
I am not a C++ guru but looking through the code I find no effort made to perform any sorting at all for the following categories:

Applet Manuals
Hardware
Control Center Modules
Konqueror Plugins

Seems all we need do is add a sort command?
Comment 2 Michele Calgaro 2014-03-07 02:20:32 CST
Darrell, I haven't looked at this but I have a question.
Does "resorting" only involves modifying text files or involves modifying code?
In the first case, could you do it? In the second case I will do it.
Comment 3 Darrell 2014-03-07 14:31:48 CST
I do not know. I seem to recall seeing sorting being explicitly used in one part of the code. Seems that should be the same behavior elsewhere in the code too.
Comment 4 Timothy Pearson 2014-10-04 14:05:04 CDT
This is a pretty ugly paper cut bug; somehow it has been overlooked it year after year even though it severely degrades usability.  Nice catch!

The application entries are most definitely sorted, see navigatorappitem.cpp line 153.  However, as noted, they are not sorted in a case-insensitive manner, and the other entries are not sorted at all.

All of this is fixed in GIT hash c292453 (tdebase).  I chose not to sort the root entries themselves as they seem to have a somewhat logical order that I don't want to disrupt, however if desired I can enable sorting on them as well.

Thanks for reporting!
Comment 5 Darrell 2014-10-04 18:20:13 CDT
Looks much nicer here. Thank you.