-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
Admin theme header issues #1214
Comments
thanks again for this report - the demo notice was completely out of my focus :-) Ive never tried it. |
@aterjung I completely understand that you didn't notice the demo notice. It is not a wide used option I guess. And not really a big deal. We use it as an extra reminder on the test environment that our customers are not working on the live environment. But I think it's also a small problem to solve, so that still would be nice. The overlapping search is a bigger problem. This is something our customers have actually issues with. For now we can use the old theme for those customers however. |
The search is indeed more problematic than i have expected it. There are to many extensions that add an new first level extension... The right way would be to move these to the second or level - but its not our choice. I tend to change the behavior of the search element to show only a magnifying glass icon and overlay the search input after a click. I think the search ist not so important. |
@aterjung I think the solution could be more easy. The menu is already continuing onto the next line as you can see on my last screenshot. The problem is, it isn't doing that at the right moment. A quick fix in my browser looks like: What I changed is that I added 350px padding to the right of the nav-bar. |
thats a short win :-) I will add this to the theme - its not perfect but makes it work. Can be done better later on. |
Preconditions (*)
Steps to reproduce (*)
2a. Have design/head/demonotice set to 1
2b. Resize your screen
Expected result (*)
Actual result (*)
or when there are more menu items:
There are 2 items behind the search. You can see this when I hide the search:
The text was updated successfully, but these errors were encountered: