New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

quick link index in alphabetical class list in classes.html doesn't work (Origin: bugzilla #764711) #5981

Closed
doxygen opened this Issue Jul 2, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@doxygen
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity normal in component general for ---
Reported in version 1.8.11 on platform Other
Assigned to: Dimitri van Heesch

On 2016-04-07 06:07:17 +0000, David M. Butler wrote:

We are using doxygen for a C++ project. The quick link index at the beginning of the alphabetical list of classes in classes.html is supposed to provide a convenient way to jump to classes starting with a particular letter. This feature doesn't work in release 1.8.11; clicking on a letter in the quick link index has no effect. Experimentation shows that this feature worked in release 1.8.5 but fails in release 1.8.6 and all subsequent releases, including 1.8.11.

Examination of the html source for the list shows that the quick links always refer to "#letter_<upper_case>" while the cells they are supposed to refer to are named "#letter_<lower_case>", for instance "#letter_E" and "#letter_e", respectively. Further experimentation shows that cell name is lower case if the first letter of the class name (after the prefix has been removed) is lower case and upper case if the first letter of the class name is upper case. So the quick links work if the class names begin with an upper case letter. See for instance the vtk document at http://www.vtk.org/doc/release/7.0/html/classes.html, generated by doxygen 1.8.9.1.

Examination of the release 1.8.11 C++ source in index.cpp shows that the references in the quick links are generated using:

line 1779: startLetter=getUtf8CodeToUpper(cd->className(),index);

while they cells they refer to are generated using:

line 1830: startLetter=getUtf8Code(cd->className(),index);

These will match only if the letter at index in className is already upper case.

Changing line 1830 to match line 1779 fixes the problem.

On 2016-05-15 14:15:20 +0000, Dimitri van Heesch wrote:

Confirmed. Should be fixed in the next GIT update.

On 2016-09-05 13:46:29 +0000, Dimitri van Heesch wrote:

This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.12. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen closed this Jul 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment