Skip to content

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

Merged
dietmarkuehl merged 6 commits intobemanproject:mainfrom
Andres-Salamanca:main
Feb 8, 2025
Merged

Proposal to Use Doxygen-Awesome to Modernize Doxygen's Look#125
dietmarkuehl merged 6 commits intobemanproject:mainfrom
Andres-Salamanca:main

Conversation

@Andres-Salamanca
Copy link
Member

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
Member

@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
Member

@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
Member 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