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

Make code in docs pretty again #37 #43

Closed
wants to merge 1 commit into from
Closed

Make code in docs pretty again #37 #43

wants to merge 1 commit into from

Conversation

pcolby
Copy link
Contributor

@pcolby pcolby commented Mar 9, 2018

The custom HTML stylesheet, which was generated for Doxygen 1.5.6 (a version just short of 10 years old now) is missing a lot of styles needed for modern versions of Doxygen.

Since Doxygen 1.8.2, there is now a HTML_EXTRA_STYLESHEET option that allows custom HTML stylesheets to be used in addition to Doxygen's own default doxygen.css sheet.

This change simply moves Grantlee's custom HTML stylesheet from the HTML_STYLESHEET option to the HTML_EXTRA_STYLESHEET option instead, resulting in a better render for modern versions of Doxygen (such as properly formatted code blocks), as well as increasing resilience to future changes from the Doxygen project.

The custom HTML stylesheet, which was generated for Doxygen 1.5.6 (a
version just short of 10 years old now) is missing a lot of styles
needed for modern versions of Doxygen.

Since Doxygen v1.8.2, there is now a HTML_EXTRA_STYLESHEET option that
allows custom HTML stylesheets to be used *in addtion* to Doxygen's own
default doxygen.css sheet.

This change simply moves Grantlee's custom HTML stylesheet from the
HTML_STYLESHEET option to the HTML_EXTRA_STYLESHEET option instead,
resulting in a better render for modern versions of Doxygen (such as
properly formated code blocks), as well as incresaing resilience to
future changes from the Doxygen project.
@pcolby pcolby changed the title Make code in docs pretty again #10 Make code in docs pretty again #37 Mar 9, 2018
@pcolby
Copy link
Contributor Author

pcolby commented Mar 9, 2018

Fixes #37

@steveire
Copy link
Owner

Cherry-picked to master, thanks!

@steveire steveire closed this Nov 24, 2018
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.

2 participants