Skip to content

Suggesting changes to ExploreMapSection.js #2

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.

Sign up for GitHub

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
alisoncase wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Suggesting changes to ExploreMapSection.js #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/components/ExploreMapSection.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ const ExploreMapComponent = () => {

// create object of keys that correspond to the emotional attributes and values as their respective labels
const attributeLabels = {
happinessSadness: 'Feeling',
calmAnxious: 'Anxious',
happinessSadness: 'Happiness', // change to more descriptive term that provides parallel structure
calmAnxious: 'Calmness', // change to more descriptive term that provides parallel structure
awakeTired: 'Alertness',
safety: 'Safety',
belonging: 'Belonging'
Expand Down Expand Up @@ -235,7 +235,7 @@ const ExploreMapComponent = () => {
}
]
})
mapView.ui.add(legend, 'bottom-left')
mapView.ui.add(legend, 'bottom-right') // move legend to bottom-right so it's in line with attribute selection widget

// Add a locate button to the view
const locateBtn = new Locate({
Expand Down Expand Up @@ -304,7 +304,7 @@ const ExploreMapComponent = () => {
</p>
<div className='explore-container'>
<div className='select-container'>
<label htmlFor='attribute-select'>Select Emotion:</label>
<label htmlFor='attribute-select'>Select Experience Type:</label> {/* Change label to more descriptive term matching the legend title*/}
<select
id='attribute-select'
value={selectedAttribute}
Expand Down