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][IMP] web_editor: use esc key to discard crop widget #163850

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Apr 29, 2024

Current behaviour before commit:

In website when cropper is open, pressing ESC key
hides side panel.

Desired behaviour after commit:

Now, pressing ESC works same as discard button. It closes
the cropper and prevents hiding side panel.

task-3787411


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #158543

Current behaviour before commit:

In website when cropper is open, pressing ESC key
hides side panel.

Desired behaviour after commit:

Now, pressing ESC works same as discard button. It
closes the cropper and prevents hiding side panel.

task-3787411

X-original-commit: b008d91
@robodoo
Copy link
Contributor

robodoo commented Apr 29, 2024

@fw-bot
Copy link
Contributor Author

fw-bot commented Apr 29, 2024

@adsa-odoo @dmo-odoo this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@fw-bot r+

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

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Apr 29, 2024
@C3POdoo C3POdoo added the RD research & development, internal work label Apr 29, 2024
@dmo-odoo
Copy link
Contributor

@fw-bot r+

robodoo pushed a commit that referenced this pull request Apr 30, 2024
Current behaviour before commit:

In website when cropper is open, pressing ESC key
hides side panel.

Desired behaviour after commit:

Now, pressing ESC works same as discard button. It
closes the cropper and prevents hiding side panel.

task-3787411

closes #163850

X-original-commit: b008d91
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
@robodoo robodoo added the 17.3 label Apr 30, 2024
@robodoo robodoo closed this Apr 30, 2024
@fw-bot fw-bot deleted the master-17.0-discard-cropper-on-esc-adsa-wmSw-fw branch May 14, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
17.3 forwardport This PR was created by @fw-bot RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants