Skip to content

Conversation

@owenfromcanada
Copy link
Contributor

As requested by my wife--for anyone who forgets where they were when counting things.

Basic functionality is tap-to-count, but shake-to-count can be enabled (using the same calibration as shake-to-wake). Includes a delay to avoid accidental double counting, and helpful "toast" messages when options are changed.

Note that the added Adwaita Mono font is for the Tally 5 character (𝍸) which I used as the app icon. This symbol is only implemented in a bare handful of existing fonts, but Adwaita is released under the OFL so should be fine to use.

@github-actions
Copy link

github-actions bot commented Jun 22, 2025

Build size and comparison to main:

Section Size Difference
text 380616B 1488B
data 948B 0B
bss 22536B 0B

Run in InfiniEmu

@mark9064 mark9064 added the new app This thread is about a new app label Jun 22, 2025
Copy link
Member

@mark9064 mark9064 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 sending this in :)

Code looks really good overall, just a few comments. Haven't run clang-tidy yet

@mark9064
Copy link
Member

One other thing: I'm guessing this is designed to go to 999 but it seems that the numbers are slightly cut off when showing 3 digits. Maybe they could be a bit smaller so they can keep a bit of margin even at 3 digits? I think the next font size down is a lot smaller though - what do you think?

@owenfromcanada
Copy link
Contributor Author

Good call--I mostly anticipated usage for lower values, but I pushed another commit that changes the font when it reaches 100. This way it's still large for (what I'm guessing is) the majority of use cases, but will still work well above that.

Copy link
Member

@mark9064 mark9064 left a comment

Choose a reason for hiding this comment

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

Ran clang-tidy, all looked good

@Saarsk
Copy link

Saarsk commented Jul 13, 2025

Good call--I mostly anticipated usage for lower values, but I pushed another commit that changes the font when it reaches 100. This way it's still large for (what I'm guessing is) the majority of use cases, but will still work well above that.

Hehe, I counted to 863 on my Pebble smartwatch just two days ago, with a tally counter app. 😅 So it's great that it'll not stop at 100. Four digis support would be nice too but I understand if you don't wanna do that. 😊

This is a great addition to the app suite! 🙂

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

Labels

new app This thread is about a new app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants