Krystal Won | Dashboard Project

[krystalwon]

No description provided.

[mjumbewu]

Nice-looking dashboard. A few suggestions below, and a few more in-line with your code.

• suggestion: You should move all your files to the root folder. This would make it easier to share your dashboard if you enable GitHub Pages (which you should do). Also related to GitHub Pages, you should try to keep spaces out of folder and file names; some web servers have trouble dealing with them.
• idea: It would be cool to be able to click within a state to select it in addition to having states be selectable from your select element

Also a couple a11y issues I saw using the Axe developer tools:

• Links must be distinguishable without relying on color -- this is referring specifically to your links in the About This Project section. The links should either have a higher contrast with the non-linked text around them, or should have something like an underline.
• Select element must have an accessible name -- this is in reference to your state select element. It should either be associated with a label element, or have an aria-label attribute.

[mjumbewu]

suggestion: This file is over 2 megabytes. That's not too egregious, but in this case it seems unnecessarily large. For any data asset over maybe half a megabyte that's going to by downloaded by the user's browser, I would consider whether it's necessary for the file to be so large. Sometimes it is necessary for files large (e.g., your cleaned workforce CSV is probably not going to get any smaller without splitting it up), but in the case of these boundary shapes, you can likely get away with simplifying the polygons a lot without losing anything, because the states are shown at such a low zoom level.

[mjumbewu]

praise: Nice; I like this about section.