Correct scroll length for left and up directions #52
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously, the calculation for the distance of leftward and upward scrolling resulted in the text bitmap not being scrolled completely off the display.
For leftward scrolling, this would be noticeable in the case where the bitmap of the last character in the string had populated pixels on the rightmost column. For upward scrolling, this would be noticeable in the case where the bitmap of any character in the string had populated pixels on the bottom row.
Demos
The incompletely scrolled pixels will not be distinguishable from the artifacts left by the unrelated bug #50, but can be seen if the fix from #51 is applied.
SCROLL_LEFT
Prior to the change proposed here, the following sketch would result in the two pixels from the rightmost column of the
Font_4x6
#
character bitmap would remaining on the display after scrolling:SCROLL_UP
Prior to the change proposed here, the following sketch would result in the pixel from the bottom row of the
Font_4x6
(
character bitmap remaining on the display after scrolling: