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

Timestamp overflow screen width #80

Closed
orontee opened this issue Nov 30, 2023 · 0 comments
Closed

Timestamp overflow screen width #80

orontee opened this issue Nov 30, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@orontee
Copy link
Owner

orontee commented Nov 30, 2023

Context

date_overflow

I guess the alert button may also hide the content of the current condition widget if visible.

Analysis

The Pocketbook Verse has a 6 inch screen with a rather low resolution, 1024x768 pixels (212 PPI).

Proposal

Compute the width of the timestamp string and change of date format when the string would overflow screen width.

⚠️ This will not fix the alert button hiding content of the current condition widget.

Side notes

To avoid widget overflow, the widget class should define the four usual padding properties, deduce a "content box" rectangle from the bounding box rectangle and those padding properties, and clip to that content box rectangle during paint operations.

@orontee orontee added the bug Something isn't working label Nov 30, 2023
@orontee orontee added this to the Noël 2023 release milestone Nov 30, 2023
orontee added a commit that referenced this issue Nov 30, 2023
@orontee orontee closed this as completed Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant