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 TextEdit cusor shape when mouse is held #102556

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

kitbdev
Copy link
Contributor

@kitbdev kitbdev commented Feb 7, 2025

When selecting text with the mouse and the cursor goes outside the textbox to the left or over something clickable like the gutters, the mouse cursor should not change.
Same with dragging the minimap, the cursor should always be the same while the mouse is held down.

I think dragging_selection shouldn't be set when dragging text, but changing it could affect other logic, so for now I just make sure to set it to false when dragging ends. Otherwise it could be in the wrong state. This could be seen by moving some text and immediately ctrl+click on something, it doesn't work.

@kitbdev kitbdev requested a review from a team as a code owner February 7, 2025 19:44
@AThousandShips AThousandShips added bug topic:gui cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release labels Feb 7, 2025
@AThousandShips AThousandShips added this to the 4.5 milestone Feb 7, 2025
Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

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

Needs rebase to fix CI.

@KoBeWi KoBeWi modified the milestones: 4.5, 4.4 Feb 8, 2025
@KoBeWi KoBeWi removed the cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release label Feb 8, 2025
@kitbdev kitbdev force-pushed the fix-te-cursor-shape-held branch from 4980814 to 65b7e1c Compare February 8, 2025 20:32
@Repiteo Repiteo merged commit 5f7ea83 into godotengine:master Feb 12, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Feb 12, 2025

Thanks!

@kitbdev kitbdev deleted the fix-te-cursor-shape-held branch February 12, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release topic:gui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants