Skip to content
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

HTML for multiple pages inside one source file seems to have wrong names, or wrong links #42

Closed
bjfar opened this issue Apr 30, 2018 · 4 comments
Assignees

Comments

@bjfar
Copy link

bjfar commented Apr 30, 2018

In doxygen it seems to be permitted to write code like this:

/*!
\mainpage Welcome
 - \subpage Test
*/

/*!
\page Test Title
*/

where two pages can be defined inside the one source file. This generates two html files, index.html and _test.html. However, when your script does it, it generates index.html and Test.html. This would be fine I guess, except that the link generated by "\subpage Test" is still to the page "_test.html", which doesn't exist, so the link doesn't work. So I guess you need to either change how the links are generated, or how these "extra" pages are named :).

@mosra
Copy link
Owner

mosra commented Apr 30, 2018

Oh, hi again! :)

This is due to the CASE_SENSE_NAMES option, which I didn't implement yet. In your Doxyfile it's set to NO, which causes Doxygen to generate files like _u_p_p_e_r.html for a page named UPPER, whereas I'm expecting it to be set to YES and thus doing no modifications of the output filenames (having UPPER.html).

Implement support for this option in m.css would probably be a good idea, but unfortunately I'm very busy for the next ~two days and don't have the time. As a temporary (or pernament?) solution you can set CASE_SENSE_NAMES back to YES (and then hope that you don't runto into random problems on case-insensitive filesystems).

@mosra mosra added this to TODO in Doxygen theme via automation Apr 30, 2018
@mosra mosra self-assigned this Apr 30, 2018
@bjfar
Copy link
Author

bjfar commented Apr 30, 2018

Ahh ok, I'll give it a try, thanks!

@mosra
Copy link
Owner

mosra commented Apr 30, 2018

Hm. Turns out it was due to my workaround to yet another Doxygen speciality where I thought some things were equivalent while they weren't.... and was quite easy to fix, actually. Should work in 1d7c689, can you confirm it works for you as well?

Also, forgot to say: your reports over the past days helped improve the parser stability quite a lot, so thanks for that! 👍

@bjfar
Copy link
Author

bjfar commented Apr 30, 2018

Ahh great, yeah that seems to have fixed it!

And no worries, happy to help work out the kinks, your tool is super easy to use and makes nice looking docs so I am pretty keen to get it working with our project. Seems like it is pretty much there now!

@bjfar bjfar closed this as completed Apr 30, 2018
Doxygen theme automation moved this from TODO to Done Apr 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Doxygen theme
  
Done
Development

No branches or pull requests

2 participants