SideNav header does not have enough height with long descriptions

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

956 stars 108 forks source link

SideNav header does not have enough height with long descriptions #129

Open prince-chrismc opened 7 months ago

prince-chrismc commented 7 months ago

Firstly I really appreciate this project, makes docs so much more approachable for users ❤️

I ran into a tiny bug, the fixed height variable used for the main content header and side nav can sometimes conflict. For my case the description is longer then expected such that #top is using --top-height: and is cutting the search bar. Adjusting to 150px make the padding on main content to big.

Links to my copy running :) https://prince-chrismc.github.io/jwt-cpp/ https://github.com/prince-chrismc/jwt-cpp/commit/b97eb1be7164fe604f697bec291580adb7534b0e

Default

Shorter text

jothepro commented 7 months ago

Hm, good point! I wasn't aware of that side effect. I'm not sure if I would consider it a bug, but it definitely is not perfect. Maybe it makes sense to decouple the contents top height from the sidebar variable. 🤔

I'll keep the issue open and think about it, feel free to contribute ideas in the meantime! 😊

prince-chrismc commented 7 months ago

Thanks! I will try and fix it with a CSS overload and report back, playing in chrome its very likely fixable from the user side :)

prince-chrismc commented 7 months ago

Here's my override.css https://github.com/prince-chrismc/jwt-cpp/blob/f7dedd0ac697f96a7a4b60ac01132e753c82e912/docs/overrides.css#L1-L17 which fixed it for my needs