Skip to content

Shows selected Watch face page first - #1872

Merged
mark9064 merged 7 commits into
InfiniTimeOrg:mainfrom
SteveAmor:watchSettingScreen
Oct 24, 2026
Merged

mark9064 merged 7 commits into
InfiniTimeOrg:mainfrom
SteveAmor:watchSettingScreen

Conversation

@SteveAmor

Copy link
Copy Markdown
Contributor

When you enter the watch face menu, it will take you to the second page if your current watch face is the one that is active. I thought it was a bit odd that the first page is shown with no radio buttons selected and you scroll down the the second page.

@github-actions

github-actions Bot commented Oct 4, 2023 •

Copy link
Copy Markdown

Build size and comparison to main:

Section Size Difference
text 393344B 16B
data 936B 0B
bss 22920B 0B

Run in InfiniEmu

@SteveAmor

Copy link
Copy Markdown
Contributor Author

Before
InfiniSim_2023-10-04_202745
After
InfiniSim_2023-10-04_203105

Comment thread src/displayapp/screens/settings/SettingWatchFace.cpp Outdated
@mark9064 mark9064 added the UI/UX User interface/User experience label Jul 5, 2025
@mark9064 mark9064 added this to the 1.17.0 milestone Jul 5, 2025
BurninTurtles added a commit to BurninTurtles/InfiniTimeBT that referenced this pull request May 25, 2026
* Shows selected watch face page first

* Update SettingWatchFace.cpp

* Update SettingWatchFace.cpp

---------

Co-authored-by: Steve Amor <steveamor@users.noreply.github.com>
Comment thread src/displayapp/screens/settings/SettingWatchFace.cpp Outdated
@mark9064
mark9064 merged commit 2d1aa29 into InfiniTimeOrg:main Oct 24, 2026
6 of 7 checks passed
@mark9064

Copy link
Copy Markdown
Member

Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI/UX User interface/User experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants