Skip to content
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

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

Open
thorndeux opened this issue Dec 8, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@thorndeux
Copy link

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 jothepro added the enhancement New feature or request label Dec 9, 2022
@jothepro
Copy link
Owner

jothepro commented Dec 9, 2022

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
Copy link
Author

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. 😁

@miguno
Copy link

miguno commented Jan 27, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants