jothepro / doxygen-awesome-css

Custom CSS theme for doxygen html-documentation with lots of customization parameters.
https://jothepro.github.io/doxygen-awesome-css/
MIT License
956 stars 108 forks source link

When two or more classes are in one header file, only the last class's code source with line number. #128

Closed traversebitree closed 7 months ago

traversebitree commented 7 months ago

Describe the bug When two or more classes are in one header file, only the last class's code source with line number. If use the original doxygen, no problem (Every line with a line number).

Expected behavior Every class in the header file should have a line number each line.

Screenshots image

To Reproduce Just a simple header file, with two or more classes declaration in it.

Version Windows 11, 64 bit Doxygen 1.9.8 doxygen-awesome-css 2.2.1

ilg-ul commented 7 months ago

I face the same issue, line number are shown only for the last class in the file.

Screenshot 2023-11-19 at 12 12 17

If I remove the doxygen-awesome-css/doxygen-awesome.css file from HTML_EXTRA_STYLESHEET, the whole file is ok:

Screenshot 2023-11-19 at 12 13 01 1

The file used in these images is detail.h, but the same happens to all files in my projects.

Any idea how to fix this?

macOS x64 Sonoma 14.1 Doxygen 1.9.8 doxygen-awesome-css 2.2.1

ilg-ul commented 7 months ago

@jothepro, any chance to investigate this bug?

My sites went public and people generally liked them, but I already got questions about the missing line numbers... :-(

jothepro commented 7 months ago

I think I found the problem. Can you verify that https://github.com/jothepro/doxygen-awesome-css/commit/c064ec143df9aa58b6234710e2fffd76445bf086 fixes the problem?

ilg-ul commented 7 months ago

I confirm that the line numbers now show on my site.

Could you make a new release so I can pick the commits? (2.2.2 in package.json if since 2.2.1 there were only bug fixes, or 2.3.0 if there were any additions to the api)

jothepro commented 7 months ago

Thx for the feedback! I will need to do some more checks to make sure that the change is backwards compatible with older versions of Doxygen, then I will make a release! 👍

ilg-ul commented 7 months ago

BTW, your cool theme was highly appreciated by those who reviewed my sites:

Thank you!