Skip to content

[feat] Display the date of first occurence - #100

Open
surli wants to merge 1 commit into
albertvaka:mainfrom
surli:feat-display-first-occurence
Open

[feat] Display the date of first occurence#100
surli wants to merge 1 commit into
albertvaka:mainfrom
surli:feat-display-first-occurence

Conversation

@surli

@surli surli commented Dec 29, 2025

Copy link
Copy Markdown

Hi,

I tried to hack a bit the app to be able to display the date of first occurrence of a data in a counter: it's often useful for me to properly analyze the data (e.g. why do I have so few occurrences of this counter in March?! Ah.. I started to count that in 15th of March...)

I don't know how you handle evolution of translations through time in the app: maybe it would be better to create new translation keys since I added a new param, since I can't properly transform all existing translations?

Also I'm not super happy of just displaying the date, I guess I should call some helper to format it properly, but idk if there's a lib for it already or if I should just use some standard formatter? (not used to hack android apps :))

@albertvaka

Copy link
Copy Markdown
Owner

I don't think it's a good addition to use the (very limited) real state in the UI to always show something that is for the most part only useful when you created a counter recently.

I personally think we don't need any indication for this, since it's pretty obvious looking at the graph that there is no data at the beginning, but if we were to add something I would only show it when there isn't a full day/week/month/year of data. Maybe just something like "partial data" somewhere if it fits, or even something more subtle like showing the number in italics or something.

@surli

surli commented Dec 29, 2025

Copy link
Copy Markdown
Author

that is for the most part only useful when you created a counter recently

Well it depends how you define recently: I generally use counter stats of the full year so on my side I need the data for any counter less old than a year basically.

But I agree with you: it's only useful in the different stats screens if the date is less than a year/month/week/day etc, so indeed I could use this as a condition. I'll check how to do it.

Now regarding where to put the info I'd keep it simple but if you think it would be better to move it elsewhere let me know, but I might need a bit of guidance then :)

@albertvaka

albertvaka commented Jan 2, 2026

Copy link
Copy Markdown
Owner

I think something subtle like showing the total count number in italics might be good. It's not super obvious what it means, but since it's not a basic/required mechanic to use the app, I think it's okay. It also doesn't need translations :D

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.

2 participants