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

[IMP] base: prevent focus on search input in settings #42661

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

dharmrajsinh-jhala
Copy link
Contributor

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses seen 🙂 labels Jan 3, 2020
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Jan 3, 2020
@C3POdoo C3POdoo added the RD research & development, internal work label Jan 3, 2020
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jan 3, 2020
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 6, 2020
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Jan 15, 2020
adr-odoo pushed a commit to odoo-dev/odoo that referenced this pull request Jan 16, 2020
Previously when we opened settings, the foucs jumped to the search input.

For better usability, this commit prevents from happening so in mobile
devices.

TaskID: 2150992
Closes: odoo#42661
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jan 16, 2020
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 17, 2020
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 17, 2020
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 17, 2020
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Jan 28, 2020
Previously when we opened settings, the foucs jumped to the search input.

For better usability, this commit prevents from happening so in mobile
devices.

TaskID: 2150992
Closes: odoo#42661
The previous style defined on the latter removed the padding which breaks
the style when several buttons follow each other.

This commit should fix the style while maintaining the intended behaviour,
a vertical alignment between the button and the text inside the pane.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants