UX improvement: Grouping separator in one setting

emavgl / oinkoin

Oinkoin is a flutter app for helping you managing your expenses. No internet required.

GNU General Public License v3.0

165 stars 11 forks source link

UX improvement: Grouping separator in one setting #18

Closed DSiekmeier closed 1 year ago

DSiekmeier commented 1 year ago

First, thanks for this promising app. I have a small recommendation for the Settings dialog.

Instead of using two separate settings for the Grouping separator ("Activate / Deactivate" and the Dropdown menu) it would be more intuituve if

the Dropdown list is visible only when "Use 'Grouping separator'" is active

There is only a single dropdown with the entries "none", "dot", "space", ....

emavgl commented 1 year ago

Hi @DSiekmeier, I am happy you like the app :) and thanks for the feedback, it is really a good point. I would go for the first option. :)

DSiekmeier commented 1 year ago

You could add the final touch to this feature with a preview of the result...

emavgl commented 1 year ago

Option 1 in the new version: https://github.com/emavgl/oinkoin/releases/tag/1.0.26 Preview will also be nice, but I think I need to rethink a bit where to put it. I will keep in mind if I end up reworking that page. For now I am closing this issue. Thanks for the feedback.