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

Doxygen: simplify logic, remove vendored header/footer/css files #738

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ManifoldFR
Copy link
Contributor

Following the discussion on my issue #737, here is my proposed change.

I will propose an update to the vendored MathJax in a subsequent PR.

Resolves #737

cc @nim65s @jmirabel

@jmirabel
Copy link
Collaborator

Is it supported to use doxygen without providing a header, footer and css ?
In the past, it used not to be the case.

@ManifoldFR
Copy link
Contributor Author

ManifoldFR commented Jan 15, 2025

Is it supported to use doxygen without providing a header, footer and css ? In the past, it used not to be the case.

Yes! From what I'm seeing in the docs:

The HTML_HEADER tag can be used to specify a user-defined HTML header file for each generated HTML page. If the tag is left blank Doxygen will generate a standard header.

I'm not aware that it was ever not the case however. I started looking around in the changelog and at some resolved issues on doxygen's repo from like over a decade ago (like doxygen/doxygen#4079), and I'm not finding anything.

@ManifoldFR
Copy link
Contributor Author

There is a mention of a default CSS stylesheet in the changelog dating back to 1.5.7 (in 2008)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Updating doxygen.cmake behaviour (remove old files?)
4 participants