-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[10.0][IMP][web_responsive] Make search bar sticky #633
[10.0][IMP][web_responsive] Make search bar sticky #633
Conversation
Can you put some screenshots of before and after to visually check? |
@pedrobaeza Yes! 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea @tarteo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just noticed that sticking the top bar in phones does not allow the user to reach some menu entries, but it's not this PR's fault, so OK for me.
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
* Initial commit * Initial commit
This PR makes the search bar (.o_control_panel) 'stick' to the secondary (top) menu bar (only on big screens).
Before:
After: