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 caret movement at start of wrapped lines #91978

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

kitbdev
Copy link
Contributor

@kitbdev kitbdev commented May 15, 2024

Fixes moving caret to start of wrapped line with up/down moving to after the first character.
Fixes setting last fit at start of wrapped line acting as the end of the previous wrap. This caused the caret to jump to the end of the previous/next line instead of moving straight up or down.

Removed some dead code in TextEdit::get_line_column_at_pos.
Added tests.

@kitbdev kitbdev requested review from a team as code owners May 15, 2024 13:08
@KoBeWi KoBeWi added this to the 4.3 milestone May 15, 2024
@kitbdev kitbdev force-pushed the select_first_wrapped_char branch from b6db4ab to 352c9ed Compare June 6, 2024 20:05
@kitbdev
Copy link
Contributor Author

kitbdev commented Jun 6, 2024

Rebased to fix conflict.

@KoBeWi KoBeWi modified the milestones: 4.3, 4.4 Jul 30, 2024
@kitbdev kitbdev changed the title Fix selecting start of wrapped line with mouse Fix selecting start of wrapped line with mouse or keyboard Aug 10, 2024
@kitbdev kitbdev changed the title Fix selecting start of wrapped line with mouse or keyboard Fix TextEdit caret movement at start of wrapped lines Aug 16, 2024
@kitbdev kitbdev force-pushed the select_first_wrapped_char branch from 352c9ed to 93a81dd Compare August 16, 2024 19:54
@akien-mga akien-mga merged commit 18f3bb7 into godotengine:master Aug 16, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks!

@kitbdev kitbdev deleted the select_first_wrapped_char branch August 17, 2024 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants