Skip to content

Feedback from Marek Boulerice #65

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

Open
MarekB19 opened this issue Mar 9, 2025 · 1 comment
Open

Feedback from Marek Boulerice #65

MarekB19 opened this issue Mar 9, 2025 · 1 comment

Comments

@MarekB19
Copy link

MarekB19 commented Mar 9, 2025

Hi Team! First of All I just wanted to say great job! This is a great dashboard, Really visually appealing and a very interesting topic! I really enjoyed clicking through and discovering all the features you guys included!

I have a few pieces of feedback, hopefully they're not too much of a pain to implement! An I apologize if theres overlap with what has already been written by other reviewers!

  1. First, the color scale on the map was a bit confusing to me. It seems to vary as we change from CD to CD. I wonder if it might be possible to keep a fixed scale as we select divisions?

  2. Scrolling through the census division is a bit of a pain. I know our team had a similar struggle with counties on a US map. We resolved it by having 2 filters, one for the state level and one for the county. Maybe your team could implement something similar? First a province filter then a division filter?

  3. Small nitpick about the violin plots. The Legend seems to be cut off, so I wasn't sure at first what the blue vs red lines represented. It took me a while to realize the violin plots were for the whole country and not the selected division. Would there be some way to easily indicate what the distribution is referring to? Otherwise though the plot is a great Idea, I found the comparison between the county and the overall average to be very insightful

  4. I really like how you guys integrated the map! great work. One other small nitpick about the tooltip window that pops up when mousing over divisions. Having the name of the division is great. It might be possible to make the text showing average commute time a bit more readable? the current version has a small typo (extra r in current), and the words "current filters" doesn't give the user much useful information.

  5. This may be just an issue with my computer, but i found that the app was quite slow to update after making inputs. This makes it hard to compare quickly between divisions. I'm not sure if there would be some way to optimize the backend side to make it a but more usable? I realize this is quite a hard task, just reporting on my user experience!

Overall though, great work, the plot seems very cohesive and informative!

@derekrodgers
Copy link
Collaborator

  • Fix the map's colour scale
  • Add a province filter
  • Ask Joel how to get the violin plots to be responsive in width
  • Typo in map tooltip
  • Look at additional performance improvements

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

No branches or pull requests

2 participants