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] web_editor: use esc to discard transform on image #160071

Closed

Conversation

adch-odoo
Copy link
Contributor

@adch-odoo adch-odoo commented Apr 1, 2024

Desired behavior after PR is merged:

I. This PR implements a functionality where pressing the Escape key
removes the transform widget from the image.
II. Also it ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

@robodoo
Copy link
Contributor

robodoo commented Apr 1, 2024

Pull request status dashboard.

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 1, 2024
@adch-odoo adch-odoo force-pushed the 17.0-discard-transform-on-esc-adch branch from b57c04c to 82916cd Compare April 3, 2024 07:30
@adch-odoo adch-odoo force-pushed the 17.0-discard-transform-on-esc-adch branch 3 times, most recently from e1a0003 to fe892a8 Compare April 23, 2024 07:25
@adch-odoo adch-odoo force-pushed the 17.0-discard-transform-on-esc-adch branch 3 times, most recently from 9236de6 to 1a6db6b Compare May 1, 2024 09:34
@deso-odoo deso-odoo marked this pull request as ready for review May 14, 2024 06:44
@C3POdoo C3POdoo requested review from a team May 14, 2024 06:46
@dmo-odoo
Copy link
Contributor

Runbot seems to be red for this PR. You may want to rebase and force push the branch.

Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151
@deso-odoo deso-odoo force-pushed the 17.0-discard-transform-on-esc-adch branch from 1a6db6b to 4cec5b9 Compare May 21, 2024 09:52
@deso-odoo
Copy link
Contributor

@dmo-odoo Rebased the branch, runbot is green 🟢

@dmo-odoo
Copy link
Contributor

@robodoo rebase-ff r+

@robodoo
Copy link
Contributor

robodoo commented May 21, 2024

Merge method set to rebase and fast-forward.

robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: #160071
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

closes #160071

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: #160071
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

closes #160071

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: #160071
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

closes #160071

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: #160071
robodoo pushed a commit that referenced this pull request May 21, 2024
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

closes #160071

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
@robodoo
Copy link
Contributor

robodoo commented May 22, 2024

@adch-odoo @dmo-odoo staging failed: ci/runbot (view more at https://runbot.odoo.com/runbot/build/62803602)

@deso-odoo
Copy link
Contributor

@robodoo retry

@robodoo
Copy link
Contributor

robodoo commented May 22, 2024

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

@dmo-odoo
Copy link
Contributor

@robodoo retry

robodoo pushed a commit that referenced this pull request May 23, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: #160071
@robodoo robodoo closed this in 463c248 May 23, 2024
@fw-bot
Copy link
Contributor

fw-bot commented May 27, 2024

1 similar comment
@fw-bot
Copy link
Contributor

fw-bot commented May 28, 2024

lohwswilson pushed a commit to lohwswilson/odoo that referenced this pull request Jun 5, 2024
Purpose of this commit:

This commit implements a functionality where pressing the 'Escape' key
removes the transform widget from the image.

task-3839151

Part-of: odoo#160071
lohwswilson pushed a commit to lohwswilson/odoo that referenced this pull request Jun 5, 2024
Purpose of this commit:

This commit ensures that pressing any character key does not result in the
removal of the image when the transform widget is active.

task-3839151

closes odoo#160071

Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
@fw-bot fw-bot deleted the 17.0-discard-transform-on-esc-adch branch June 6, 2024 09:46
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