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

InputText word/line selection mode tests #61

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ronak69
Copy link

@ronak69 ronak69 commented Oct 4, 2024

First draft of tests as requested in ocornut/imgui#8032.

CalcTextSize() gives bouding rects. So, had to hack together index_pos() to get coords for individual chars in multiline text.

widgets_inputtext_selection_modes_0000.mp4

Add an optional bool parameter `hold` to specify whether to submit the
final mouse up event.
@ocornut
Copy link
Owner

ocornut commented Oct 8, 2024

Thanks a lot!
I am a little swamped right now but this looks great and is exactly the sorts of thing I needed for ocornut/imgui#8032

@ronak69
Copy link
Author

ronak69 commented Oct 8, 2024

cool. how do you think a regression here will look like? are you concerned about unrelated code breaking selection, or changes to the selection code itself breaking selection?

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.

2 participants