Wrap note texts by bjorkert · Pull Request #416 · loopandlearn/LoopFollow
Conversation
This PR solves the issue where texts are to long to fit on the screen, as described in #405
Now note texts are wrapped at 40 characters.
Code Review
The changes look appropriate.
Test
Enter a very long note with a very long word into the Nightscout URL for a LoopFollow test phone.
Take screenshot before and after building with this PR. Also take a screenshot after navigating to the Nightscout URL from within LoopFollow.
From Left to Right:
- before using this PR
- after using this PR
- view from Nightscout URL
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approve from code review and test
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
2 participants