-
-
Notifications
You must be signed in to change notification settings - Fork 346
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
Switch doxygen documentation theme #1546
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1546 +/- ##
==========================================
- Coverage 70.50% 70.50% -0.01%
==========================================
Files 376 379 +3
Lines 59081 59091 +10
Branches 21222 21228 +6
==========================================
+ Hits 41657 41661 +4
- Misses 14344 14351 +7
+ Partials 3080 3079 -1
... and 15 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
I added a couple of tweaks that became possible with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @ischoegl! This is definitely an improvement over the default styling. While I suspect there are some additional tweaks we may want to make in time, there's just one issue that I think should make now, noted below.
Changes proposed in this pull request
Use modern doxygen-awesome-css theme (as mentioned in Cantera/enhancements#115 (comment)) and put details on top via customized xml layout
Rendered Webpage
vs. current theme (after #1543)
Relevant Issues
Somewhat relevant to Cantera/enhancements#115
Checklist
scons build
&scons test
) and unit tests address code coverage