Skip to content

Suggesting changes to ExploreMapSection.js - #2

Open
alisoncase wants to merge 1 commit into
afenix:mainfrom
alisoncase:main
Open

Suggesting changes to ExploreMapSection.js#2
alisoncase wants to merge 1 commit into
afenix:mainfrom
alisoncase:main

Conversation

@alisoncase

Copy link
Copy Markdown

This script was well-organized and easy to follow. Congrats! When I tested the app, my main suggestion is to re-configure the Explore Map section so the legend and attribute selection widget are better aligned. When I first explored this map, it took me a some time to figure out that "Select Emotion" was linked to the legend, both because they are located on either ends of the map and because the title of the attribute selection widget didn't match the legend title. My suggested changes:

  1. Move the legend to the bottom-right so it's on the same side of the map as the attribute selection widget. You also might consider placing both of these widgets in the bottom right, with the selection widge stacked just above the legend.
  2. Change the title text in the selection widget from "emotion" to "experience type" to better match the legend.
  3. Change the names of some attributes to provide for parallel stucture in the list (i.e., happiness, calmness, alertness vs. feeling, anxious, calmness).

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

Successfully merging this pull request may close these issues.

1 participant