emavgl / oinkoin

Oinkoin is a flutter app for helping you managing your expenses. No internet required.
GNU General Public License v3.0
139 stars 8 forks source link

Add option to switch months etc in graph view #116

Open hakaishi opened 5 months ago

hakaishi commented 5 months ago

Description of current state

Once you open the graph view, you have to go back to the home screen and then choose a month, date range etc and then open the graph view again.

Proposal

It would be nice if you could change the year/month/date range directly inside the graph view. For example, by adding the calendar button from the main view to the top right of the graph view (right aligned to the title for example).

emavgl commented 5 months ago

Good point, thanks for taking the time to write this issue.

I am not sure about this to be honest. I like that the homepage is the only source of truth for the records, and the statistic page it is designed to apply some "action" on the records showing in the homepage. Giving the possibility to change the records scope in two different places would in my opinion make it a bit more confusing. Would it also change the scope of the records on the homepage, for example?

hakaishi commented 5 months ago

Ah. I see your point.

In my opinion, it would make sense to have a consistent timeline. That would mean that the time range of the "record view" would also change.

If it were me, I'd differentiate the main view and the "record view".

For example:

Just an idea. Not that I would expect that you restructure your app.

newhinton commented 4 months ago

A suggestion i wanted to make, which is slightly related, is to add a "next" and "previous" button to the overflow menu around the range-selector. This way, switching the range becomes more easy in the "main" view for what i assume are the most common usecases like switching months.