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

WatchFacePineTimeStyle: Fix conditional in weather display #1965

Merged
merged 1 commit into from
Jan 14, 2024

Conversation

vkareh
Copy link
Contributor

@vkareh vkareh commented Jan 13, 2024

Since returning a valid weather is always considered an updated value, if the current weather is empty, the face will attempt to display the temperature and icon as empty values, rather than clearing the labels.

Since returning a valid weather is always considered an updated value,
if the current weather is empty, the face will attempt to display the
temperature and icon as empty values, rather than clearing the labels.
Copy link

github-actions bot commented Jan 13, 2024

Build size and comparison to main:

Section Size Difference
text 369768B -16B
data 940B 0B
bss 63516B 0B

Copy link
Member

@FintasticMan FintasticMan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for finding this!

@kieranc kieranc merged commit 264b5be into InfiniTimeOrg:main Jan 14, 2024
7 checks passed
mark9064 pushed a commit to mark9064/InfiniTime that referenced this pull request Jan 25, 2024
…eOrg#1965)

Since returning a valid weather is always considered an updated value,
if the current weather is empty, the face will attempt to display the
temperature and icon as empty values, rather than clearing the labels.
@FintasticMan FintasticMan added this to the 1.15.0 milestone Jan 28, 2024
lasnikr pushed a commit to lasnikr/LasnikrInfiniTime that referenced this pull request Mar 5, 2024
…eOrg#1965)

Since returning a valid weather is always considered an updated value,
if the current weather is empty, the face will attempt to display the
temperature and icon as empty values, rather than clearing the labels.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants