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

Proposal to Use Doxygen-Awesome to Modernize Doxygen's Look #125

Merged
merged 6 commits into from
Feb 8, 2025

Conversation

Andres-Salamanca
Copy link
Collaborator

I added the stylesheet from the Doxygen-Awesome project. This project provides a highly customizable CSS stylesheet designed to freshen up Doxygen's default style.

  • Modern Design: Offers a cleaner and more user-friendly design compared to the default style.
  • Customizability: The stylesheet allows further adjustments to align with specific branding or aesthetic preferences.

Here’s a preview of how Doxygen could look with this stylesheet applied:
image
image

The stylesheet supports further tweaks through CSS variables, such as:

--primary-color
--primary-dark-color
--primary-light-color

--page-background-color
--page-foreground-color
--page-secondary-foreground-color

and more others : Customization.

Copy link
Collaborator

@dietmarkuehl dietmarkuehl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks good to me. I think may be reasonable to include an appropriate SPDX-License-Identifier: <license-identifier> into all files, especially as their license is different from the other files.

Copy link
Collaborator

@dietmarkuehl dietmarkuehl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems review-dog is upset about trailing spaces in some of the files (and I should look at the issue with the "github token").

@Andres-Salamanca
Copy link
Collaborator Author

I added the SPDX-License-Identifier: MIT License to the CSS and JS files and used prettier to automatically fix whitespace issues. But some whitespace-related changes are still being flagged

@dietmarkuehl dietmarkuehl merged commit 72262a2 into bemanproject:main Feb 8, 2025
1 check passed
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