Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some small improvements #7

Open
2 of 5 tasks
DJCrashdummy opened this issue Jun 6, 2020 · 6 comments
Open
2 of 5 tasks

some small improvements #7

DJCrashdummy opened this issue Jun 6, 2020 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@DJCrashdummy
Copy link
Contributor

DJCrashdummy commented Jun 6, 2020

  • it would be nice to add the ability to set a persistent default country, which perhaps also loads the charts by default.
  • if a default country is chosen, a simple button in the top bar to quickly switch between the default country and the global stats may be interesting.
  • also the possibility to mark more than one country as favorites, which are e.g. shown on the top of the select page may be interesting. - especially if you live in a smaller country or near a border and regularly want to check the situation around your own country.
  • also the possibility to set the default view (total|daily) for the charts may be interesting. - because in some countries resp. after some time the total chart does not change that much.
  • the icing on the cake would be to use separators for the numbers according to the devices regional settings(!) to grasp them more easily.
@DJCrashdummy DJCrashdummy changed the title some little improvements some small improvements Jun 6, 2020
@GabrielTavernini GabrielTavernini self-assigned this Jun 6, 2020
@GabrielTavernini GabrielTavernini added the enhancement New feature or request label Jun 6, 2020
@GabrielTavernini
Copy link
Owner

I don't have much time to work on this project right now, but as soon as school and exams are over I'll try to implement some of the proposed features.

In the meantime, feel free to submit pull requests for any of those features. I'll try to be as quick as possible to review and merge them.

@DJCrashdummy
Copy link
Contributor Author

I don't have much time to work on this project right now, but as soon as school and exams are over I'll try to implement some of the proposed features.

no pressure! these are just some suggestions which came into my mind while trying your nice app. - take the time you need to study and pass your exams... best wishes to you! 🤞

In the meantime, feel free to submit pull requests for any of those features. I'll try to be as quick as possible to review and merge them.

besides i'm also really busy the last weeks (hence the late reply - sorry!), i'm unfortunately not really a dev, i'm just a bug hunter and only writing & fiddling around with scripts. 😒 else i'm a FOSS enthusiast, and thankful that skilled devs like you write awesome FOSS... thank you! 👏

@GabrielTavernini
Copy link
Owner

I wanted to let you know that in the latest release (2.0) separators have been added and going forward I'll try to implement some of the other features that you suggested.

@WildOrangutan
Copy link

I may leave you a PR or two, if you don't mind. Are you currently working on any of these features?

@GabrielTavernini
Copy link
Owner

Pull requests are more than welcome, feel free to do as many as you want. At the moment I'm not working on any of those features. I've added the button for a setting page but I didn't have time to add any of the features.

Months ago I thought about how to implement these features and my idea was to have the default country and view fields in the settings page, and the ability to set favorites in the country selection page (maybe activated by a floating button).
Anyway, feel free to do whatever you think is better and let me know if you have any questions about the code.

@WildOrangutan
Copy link

Ok, I will start by adding two minor features:

  • Add selected country persistence, so previously selected country is saved between app restarts
  • Load chars by default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants