search

jothepro / doxygen-awesome-css

Custom CSS theme for doxygen html-documentation with lots of customization parameters.
https://jothepro.github.io/doxygen-awesome-css/
MIT License
981 stars 112 forks source link

Fix code font-size in function declarations #42

Closed nilsvu closed 2 years ago

nilsvu commented 2 years ago

The font size in function declarations was rendered in --page-font-size before. This was a bit large and inconsistent with code in other places, so this change fixes it to --code-font-size.

jothepro commented 2 years ago

I kind of prefer the current font-size over what you propose, it feels like the visual weight is more balanced if the font is a bit larger...

Maybe this should be made configurable by a separate variable instead? 🤔

Thanks for pointing out the typos: https://github.com/jothepro/doxygen-awesome-css/commit/a309d635604a74b2775c7f74958898ed6e36353c

nilsvu commented 2 years ago

I found the font size too large particularly for long function declarations, e.g. in template code. The real problem is that horizontal scroll bars appear, so the function declaration is obscured. Making the font size smaller fixed this issue partially. Here's an example with the font size from this PR, which is better than the larger font size, but not perfect:

Bildschirmfoto 2021-09-29 um 11 47 20

jothepro commented 2 years ago

FYI I've decided to not change the font-size of the function declarations for now. But I've added a CSS variable --memname-font-size for convenience that allows to configure the size independently.

Let me know if that works for you! :)

nilsvu commented 2 years ago

Sounds good 👍