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

[FW][FIX] web: update search view menu on click IME suggestion menu #44131

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Jan 28, 2020

Before this commit, when typing something in the search view in
Japanese, and then clicking on a suggestion from the IME dropdown
menu, the search view menu did not update with selection.

Steps to reproduce:

  • Enable Japanese IME in hiragana mode;
  • Type "test" in search view;
  • Soft-select another suggestion item, e.g. "テスト";
  • Double-click on suggestion item "test";

=> The search view menu still detects "テスト".

As a result, clicking on any of these search view suggested filters
picks "テスト" instead of "test".

This commit fixes the issue by updating search menu when clicking
in a suggestion in the IME menu.

opw-2061590

Forward-Port-Of: #44094
Forward-Port-Of: #43723

Before this commit, when typing something in the search view in
Japanese, and then clicking on a suggestion from the IME dropdown
menu, the search view menu did not update with selection.

Steps to reproduce:

- Enable Japanese IME in hiragana mode;
- Type "test" in search view;
- Soft-select another suggestion item, e.g. "テスト";
- Double-click on suggestion item "test";

=> The search view menu still detects "テスト".

As a result, clicking on any of these search view suggested filters
picks "テスト" instead of "test".

This commit fixes the issue by updating search menu when clicking
in a suggestion in the IME menu.

opw-2061590

X-original-commit: 1bddc44
@fw-bot
Copy link
Contributor Author

fw-bot commented Jan 28, 2020

This PR targets saas-13.1 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added forwardport This PR was created by @fw-bot CI 🤖 Robodoo has seen passing statuses labels Jan 28, 2020
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Jan 28, 2020
@alexkuhn
Copy link
Contributor

Continuation from 12.0 with tests: #44112

@alexkuhn alexkuhn closed this Jan 29, 2020
@robodoo robodoo added closed 💔 and removed CI 🤖 Robodoo has seen passing statuses labels Jan 29, 2020
@alexkuhn alexkuhn deleted the saas-13.1-11.0-fix-searchview-composition-aku-bur--fw branch January 29, 2020 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants