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 and test case for 1737 #1766
Conversation
New test TestEntry_CursorColumn_Wrap2 to address the bug (1737)[fyne-io#1737]
Image file as an input to verify TestEntry_CursorColumn_Wrap2
Added an extra if condition to address the edge case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fix looks good but can you use a more sensible filename for the test artifact?
Can we get 1 more set of eyes please? I was quite close to the implementation of this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add tests to ensure the fix works for each of the 3 types of wrapping; on, off, truncate. So far it appears only the first two are tested.
Description:
Moves the cursor down to next line with wrapping after the line is full
Fixes #(issue)
1737
Checklist: