Open alexYinanGu0 opened 2 years ago
Reviewer: Group 19
Persona
Frontend / layout
Is the text (plot labels / titles / subtitles) easy to read? Yes, they are easy to read.
Are the plots selected appropiated for the data? Can you think of other alternatives or improvements for the plots regarding interactivity/reactivity or plot selection? Yes, they are appropriate. For the Sales trend plot, we think it would be great to have an x-axis slider with two knobs to select the minimum and maximum years.
Was the dashboard organized in a way that was easy to read (columns / rows) or you would suggest a change in this sense? The dashboard was generally organised well. However, due to a difference in screen size, it was a bit difficult to see all the plots in the dashboard at once (see screenshot below). We think it is also worth considering organising the bar plots all in one column.
Would you prefer including tabs / navbars to improve the navigation or you think it is better to visualize everything in one screen by the nature of the content? We think the tabs in the dashboard were useful and was appropriate for the visualization.
Are the plots or other images described properly for the persona described? Yes, they were properly described.
Backend
Are the options that you can select in the widgets working fine or have you found some cases where it is not working? The x-axis slider for the Sales Trend plot worked properly, however, the x-axis starts to show decimal values (e.g. 2013.5) when the range of years is smaller. It would be great to only show the integer years. All the other widgets were working properly.
If there are interactive plots: are the labels clear and easy to understand? Yes, the labels are clear and easy to understand.
Documentation
Was the documentation easy to understand from the README.md
how to use the app? How could the documentation be improved?
Yes, the documentation was easy to understand and there was even a gif showing users how to use the tab!
Was the app intuitive to use without reading the documentation? What information could be good to include? Yes, the app was very intuitive to use. For the x-axis slider, it might be nice to add a small text saying something along the lines of "Select a year range".
Indicate something you would like to include in your own app after reviewing this one After reviewing this dashboard, we think that our group can include a tab because it is good for organizing plots.
To complete by the group
proposal.md
file where it is the description of the persona and scenario: https://github.com/UBC-MDS/video_game_sales_dashboard/blob/main/doc/proposal.mdTo complete by the group reviewing:
Persona
Frontend / layout
Backend
Documentation
README.md
how to use the app? How could the documentation be improved?Indicate something you would like to include in your own app after reviewing this one