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

Hide scaling handles of shapes when text cursor is visible. #11035

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

gokaysatir
Copy link
Contributor

Issue:

  • Open an Impress file and start editing a textbox.
  • See that handles are visible.
  • Try to re-scale the selected shape.
  • It doesn't work but the preview is shown.
  • When text editing is active, scaling is not enabled.
  • We need to hide the handles in that case.

Change-Id: I0ebf20270fc08f8553c1985456836711369407e4

  • Resolves: #
  • Target version: master

Summary

TODO

  • ...

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@gokaysatir gokaysatir requested a review from eszkadev January 28, 2025 11:09
@gokaysatir gokaysatir force-pushed the private/gokay/impress-scaling-handlers branch 2 times, most recently from 876ae46 to 13c71d3 Compare January 29, 2025 15:13
Issue:
* Open an Impress file and start editing a textbox.
* See that handles are visible.
* Try to re-scale the selected shape.
* It doesn't work but the preview is shown.
* When text editing is active, scaling is not enabled.
* We need to hide the handles in that case.

Signed-off-by: Gökay Şatır <[email protected]>
Change-Id: I0ebf20270fc08f8553c1985456836711369407e4
Modify tests accordingly.

Signed-off-by: Gökay Şatır <[email protected]>
Change-Id: I4527ad3696df66d6a624792eaf30c6c44b79936d
@gokaysatir gokaysatir force-pushed the private/gokay/impress-scaling-handlers branch from 13c71d3 to de00e84 Compare January 30, 2025 09:32
@gokaysatir
Copy link
Contributor Author

Tests are passing on local. Rebased.

@eszkadev eszkadev merged commit 875c3b0 into master Feb 3, 2025
14 checks passed
@eszkadev eszkadev deleted the private/gokay/impress-scaling-handlers branch February 3, 2025 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants