neherlab / covid19_scenarios

Models of COVID-19 outbreak trajectories and hospital demand
https://covid19-scenarios.org
MIT License
1.36k stars 352 forks source link

Improve styles and readability #83

Closed rodolfoalmeida01 closed 4 years ago

rodolfoalmeida01 commented 4 years ago

Hi! Based on issues #71 and #13 me and @arieltonglet are assessing improvements to the general user experience. We're planning on altering text, button and help icon styles to improve readability and information hierarchy on both desktop and mobile.

Later on we intend to work on the charts, but it'd be best to open a separate issue for that.

ivan-aksamentov commented 4 years ago

@rodolfoalmeida01 @arieltonglet Hello, you are most welcome!

If possible, try to split the work in small pieces to avoid huge merge conflicts. Feel free to create "work in progress [DO NOT MERGE]" kind of Pull requests. Rebase on master often. Your PRs are autodeployed on Now.sh and the bot will give you a link.

It make sense to coordinate the work with Internationalization folks in https://github.com/neherlab/covid19_scenarios/issues/56 , because their changes will be very intrusive too.

Feel free to join our new chat:

Image for the link to join the chat

Let me know if you have questions, or if you need me to organize anything (permissions, subscriptions, etc.)

Create! Improvise! 🧑‍🎨

ivan-aksamentov commented 4 years ago

If you folks have a list of TODOs in mind, let us know. We can setup a Project if you'd like too. We need to ensure that there are no huge merges.

I just merged a couple of Mobile PRs. Not sure if those align to your ideas: https://github.com/neherlab/covid19_scenarios/pull/84 https://github.com/neherlab/covid19_scenarios/pull/87

rodolfoalmeida01 commented 4 years ago

Hi, @ivan-aksamentov, thanks! We've designed the following wireframe sketch with some of the changes we propose (desktop only for now). It'd probably be best indeed to split these changes in smaller pull requests so as to avoid merge conflicts.

MacBook Pro - 1

PROPOSED CHANGES: Design

Other changes

WHAT IS NOT COVERED:

ABOUT THE SCENARIO PRESETS One thing we found confusing about the scenario presets was that they aren’t labeled as presets. Therefore, it could be confusing to be given the option to fill the “population”/“universe” box with a city’s data, after selecting “country - no mitigation” on the first field. Also, selecting "country - no mitigation" defaults to Germany, when there are other fields also being altered. Labeling it as "Germany, no mitigation, moderate/north" might convey all the changes occurring.

Let us know what you think about these proposed changes.

ivan-aksamentov commented 4 years ago

@rodolfoalmeida01 This looks wonderful! Can't wait to see these little checkboxes checked :)

Change “Population” subtitle to “Universe”. We think it makes sense, as the box contains data for hospital beds and time range. Is this term correct?

I am not sure. @rneher should know better. I think we plan to move simulation params like date range somewhere else.

One thing we found confusing about the scenario presets was that they aren’t labeled as presets. Therefore, it could be confusing to be given the option to fill the “population”/“universe” box with a city’s data, after selecting “country - no mitigation” on the first field. Also, selecting "country - no mitigation" defaults to Germany, when there are other fields also being altered. Labeling it as "Germany, no mitigation, moderate/north" might convey all the changes occurring

The general idea behind scenarios:

So yes, the word "country" in scenario names is probably wrong now that we also have regions and cities (it was not the case last week ;))

I would insist on word "scenario", and not "preset", because it is closer to the ear of an epidemiologist, plus we already have it in out brand name and new domain name (to be announced soon).

Mitigation card should be visible on the page load. It is our primary goal to convince people that infection control matters. We are planning some drastic changes to the way mitigation data is entered by user: https://github.com/neherlab/covid19_scenarios/issues/14

Note: our users are not the same as a typical web user: these are epidemiologists, biologists, physicists, but also scientists and clerks in the government, possibly practicing medics, and hospital administrators. Clarity and ability to make work done and make a hard decision is more important for them than animations and transitions.

Let me know what you think. We are looking not only to prettify styles, but also improve the user experience as a whole. So any suggestions are welcome!

Related: https://github.com/neherlab/covid19_scenarios/issues/14 https://github.com/neherlab/covid19_scenarios/issues/120

P.S. Parts of this post can make an excellent user documentation page

arieltonglet commented 4 years ago

Thank you for the thorough explanation, @ivan-aksamentov ! We'll keep the "scenarios" term, and make some adjustments to the scenarios input. As @rodolfoalmeida01 go on with this, I'm forking the project and making a draft pull request for the adjustments we are talking about here

ivan-aksamentov commented 4 years ago

@arieltonglet You are fantastic folks! Thank you. Your sketch also look like an A4 page. I wonder if later we can also have a PDF- and print-friendly layout (for "busy executives")? https://github.com/neherlab/covid19_scenarios/issues/110

ivan-aksamentov commented 4 years ago

This was implemented in #336 General followup is in #13 #71