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

Improve WTextField #140

Merged
merged 4 commits into from
Nov 16, 2021
Merged

Improve WTextField #140

merged 4 commits into from
Nov 16, 2021

Conversation

nea89o
Copy link
Contributor

@nea89o nea89o commented Oct 30, 2021

Potentially fixes #139

Functionality changes made to WTextField:

  • Introduce text scrolling
  • Fix CTRL+Movement key movement
  • General refactoring

Also deprecated the public getCaretPos method since it was public.
Introduce CTRL+(Left/Right) for deletes, selects and cursor moves.
@nea89o
Copy link
Contributor Author

nea89o commented Oct 30, 2021

This shouldn't have any breaking API changes, but at this point im a bit too tired to be sure 😅

@Juuxel Juuxel self-requested a review October 30, 2021 11:41
Copy link
Member

@Juuxel Juuxel left a comment

Choose a reason for hiding this comment

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

Thanks for this PR! I just have some minor comments. Also, it'd be great if you could revert the formatting changes since the diff is hard to read currently because of them.

@nea89o nea89o requested a review from Juuxel November 12, 2021 22:29
@Juuxel Juuxel changed the base branch from master to 1.17 November 16, 2021 20:20
@Juuxel
Copy link
Member

Juuxel commented Nov 16, 2021

Thanks for this, I'm going to clean up some rough edges here and merge it to the 1.18 pres too a bit later.

@Juuxel Juuxel merged commit 5a7f7a6 into CottonMC:1.17 Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WTextFields don't have the full functionality of a vanilla text field
2 participants