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

Sidebar-only theme: search bar gets pushed out of view by longer `PROJECT_BRIEF` #100

Open thorndeux opened 1 year ago

thorndeux commented 1 year ago

Hello doxygen-awesome-css team,

first of all, I have to congratulate you on your amazing work - the theme looks great!

I noticed a small issue with the sidebar-only theme: When the PROJECT_BRIEF exceeds one line in length, it will start pushing the search bar out of view, as the --top-height is hard-coded at 120px. While this can be easily fixed on a project-by-project basis by adjusting the value of --top-height, it would be great if the #top section could accommodate longer project briefs by default.

I am rather inexperienced with CSS, so I am not sure how best to do this (I assume you would like to avoid relying on Javascript for the base layout). Maybe use flex-items or a grid instead?

Cheers, thorndeux

jothepro commented 1 year ago

Thank you for the kind feedback! 😊

As you describe it correctly, the recommended way of avoiding an overflow of the project brief is to adjust the --top-height variable manually. The underlying reason for this workaround is that the header with project title, brief and searchbar are positioned absolutely on top of the sidebar to achieve the wanted looks. They are not sharing the same block at all, which makes it hard to allow a dynamic readjustment of the header height.

In any case I'll keep this issue open, maybe someone can come up with a solution at some point. 👍

thorndeux commented 1 year ago

Yeah, that makes a lot of sense. Trying to make it responsive is probably one of these things where the effort is not worth the result, seeing how easy it is to adjust the height manually. Still, won't hurt to leave the issue open, in case someone feels inspired. :grin:

miguno commented 1 year ago

+1, same issue here after a fresh and first installation of this project.