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

[FIX] test_themes: adapt switcher tooltip format #765

Closed
wants to merge 1 commit into from

Conversation

brboi
Copy link
Contributor

@brboi brboi commented Feb 2, 2024

This adapts the WebsiteSwitcherSystray's DropdownItem
tooltips dataset to the changed API brought in
the community repository.

Task: 3266145

@robodoo
Copy link
Collaborator

robodoo commented Feb 2, 2024

This adapts the WebsiteSwitcherSystray's DropdownItem
tooltips dataset to the changed API brought in
the community repository.

Task: 3266145
@brboi brboi changed the title [IMP] web: Dropdowns menus unification [FIX] test_themes: adapt switcher tooltip format Feb 5, 2024
@brboi brboi marked this pull request as ready for review February 6, 2024 08:59
Copy link

@aab-odoo aab-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Feb 7, 2024

I'm sorry, @aab-odoo. I'm afraid I can't do that.

@sdegueldre
Copy link
Contributor

@robodoo delegate=aab-odoo

@aab-odoo
Copy link

aab-odoo commented Feb 7, 2024

robodoo r+

robodoo added a commit to odoo/odoo that referenced this pull request Feb 7, 2024
We currently have too many different implementation of the same concern: an input with a dropdown/popover/select menu that can be customized, display a list of elements, let the user navigate with the keyboard, can be closed with escape, can be closed with an outside click. Also, the current navigation systems does not handle inputs properly.

This commit addresses that by doing multiple things:
- It tries to improve the Dropdown component with a cleaner props API while keeping its ease of use.
- The navigation has been improved to support inputs and better customization.
- The Dropdown now uses the popover/overlay service which will cause less z-index issues.

closes #137691

Related: odoo/design-themes#765
Related: odoo/enterprise#48452
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
robodoo pushed a commit that referenced this pull request Feb 7, 2024
This adapts the WebsiteSwitcherSystray's DropdownItem
tooltips dataset to the changed API brought in
the community repository.

closes #765

Task: 3266145
Related: odoo/enterprise#48452
Related: odoo/odoo#137691
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
@robodoo robodoo closed this Feb 7, 2024
@robodoo robodoo added the 17.2 label Feb 7, 2024
@sdegueldre sdegueldre deleted the master-web-unify-dropdown branch February 8, 2024 10:23
xlu-odoo pushed a commit to odoo-dev/design-themes that referenced this pull request Jun 21, 2024
This adapts the WebsiteSwitcherSystray's DropdownItem
tooltips dataset to the changed API brought in
the community repository.

closes odoo#765

Task: 3266145
Related: odoo/enterprise#48452
Related: odoo/odoo#137691
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants