-
Notifications
You must be signed in to change notification settings - Fork 219
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
add justify= to .string() and deprecate right= #469
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Tony Robinson <tony@tonyRobinson.com>
bump Yes, I know I screwed up with the last PR (#459) getting everything into one merge. However, I don't think this is a complex issue and I do think I addressed all concerns. Let me try to make a strong case for this PR. Screen drawing is slow, if we call fill() and start again it's visible. If we want to update just part of the screen with text then we really don't want to blank it and write an unknown length, we really want to make one call with a width and it simply overwrites what's there. So all calls to .string() really should have width= set and we can't do that at the moment if we want left justification. So we need this PR. I have a watch face which also displays the tide times for holiday use. When I switch location it used to blank the screen and redrew at a new location which was very noticeable. I've just updated it so that it redraws on the old screen with the new location data, it looks so much nicer. Unfortunately I don't have left justification - it's waiting on this PR (not that I'd be offended if nobody wanted my tide times - it is niche). |
Tide times...now that would be nice! I thought Daniel passed you the
torch to merge this and other PR's into the current source, or did I
miss something?
On 2023-11-09 06:44, Tony Robinson wrote:
bump
Yes, I know I screwed up with the last PR (#459 [1]) getting everything into one merge. However, I don't think this is a complex issue and I do think I addressed all concerns.
Let me try to make a strong case for this PR. Screen drawing is slow, if we call fill() and start again it's visible. If we want to update just part of the screen with text then we really don't want to blank it and write an unknown length, we really want to make one call with a width and it simply overwrites what's there. So all calls to .string() really should have width= set and we can't do that at the moment if we want left justification.
I have a watch face which also displays the tide times for holiday use. When I switch location it used to blank the screen and redrew at a new location which was very noticeable. I've just updated it so that it redraws on the old screen with the new location data, it looks so much nicer. Unfortunately I don't have left justification - it's waiting on this PR (not that I'd be offended if nobody wanted my tide times - it is niche).
--
Reply to this email directly, view it on GitHub [2], or unsubscribe [3].
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
Conditional on one approval, which is missing. Let me have a look... |
@GaryM0101 Tides has been a lot of fun. I started in early September after returning from a holiday house that had a broken tide clock. I thought it would only take a couple of weeks to do something much better. Well, it is much better, just it took me a couple of months. I'll write it up in the PR, there's a fair amount to talk about. @fgaz Thanks! |
Unfortunately I got in a right mess with #459 and no longer had the confidence I was doing the right thing.
This PR addresses the known issues with #459, namely it makes minimal changes to the documentation and is a single commit.