-
Notifications
You must be signed in to change notification settings - Fork 121
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
Consider upstream integration #30
Comments
Yep, that's a good idea. I am already in contact with the Maintainer of doxygen: doxygen/doxygen#6658 Nevertheless I'd prefer to give the theme some time to incubate in this repository, with a smaller user base. And I expect bringing the theme officially to Doxygen to be very labor intensive. As long as I can achieve what I want by patching the existing theme I guess my incentive to contribute to Doxygen will be too small for it to happen anytime soon. :/ |
I like you theme very much! It is probably a very good idea to mature the theme here before integrating upstream. Indeed fewer users, but also much faster release cycle. However, I too would love to see it integrated upstream. In the end that is simpler as a user, but probably limits the number of work-arounds in the long run. |
It seems that doxygen has released a dark theme : doxygen/doxygen#8404 . I would feel that the time is right to integrate upstream. Since there is now a configuration option |
I feel like I should give you an update about my current thoughts on this topic as a maintainer: Atm I have to patch the theme for new releases of doxygen on a regular basis, which I realized is a risky strategy to go forward. There may at one point be changes that cannot be "fixed" with a few more lines of CSS overrides added to the theme. In fact this has already happened with the doxygen 1.9.5 release. I'll definitely have to evaluate strategies on how to mitigate this risk in the near future. At that point one may propose in doxygen/doxygen to add a switch to choose between the "default" and the "awesome" (🤭) theme variant. Thanks @tdegeus for the idea to make this a Any initiative from my side to create the standalone version of the theme and discuss at doxygen/doxygen how an integration would be possible will likely not happen in the next couple of months. But working towards this goal seems inevitable for the future of this theme and it will be my main focus in the mid term. |
Hi again,
Did you consider submitting your work upstream to be integrated directly into Doxygen? While I really like your solution, I'm a little concerned about incompatibilities with future Doxygen versions. Furthermore, this would allow a much larger group of people to benefit from you work. Just a thought...
The text was updated successfully, but these errors were encountered: