Skip to content
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

Jump to bottom

add legend widget #229

Closed
allisonmuise opened this issue Aug 23, 2016 · 10 comments
Closed

add legend widget #229

allisonmuise opened this issue Aug 23, 2016 · 10 comments
Assignees
@CTLocalGovTeam
Labels
Enhancement High Verified
Milestone
AGOL 5.4

Comments

@allisonmuise
Copy link
Contributor

Display of reference layers has added many features to the UI. Would be helpful to have a legend to help interpret complex maps.
@allisonmuise allisonmuise modified the milestone: Update to 4.x API Aug 23, 2016
@allisonmuise allisonmuise modified the milestones: Update to 4.x API, AGOL 5.3 Nov 23, 2016
@allisonmuise
Copy link
Contributor Author

https://geonet.esri.com/ideas/12968-option-to-add-legend-to-crowdsource-apps

@allisonmuise allisonmuise modified the milestones: AGOL 5.3, Backlog Feb 27, 2017
@allisonmuise allisonmuise modified the milestones: Backlog, AGOL 5.4, AGOL 5.3 Jun 5, 2017
@allisonmuise allisonmuise modified the milestones: AGOL 5.3, AGOL 5.4 Jun 14, 2017
@allisonmuise
Copy link
Contributor Author

allisonmuise commented Aug 15, 2017 •
edited

The UI for this might shift slightly over the course of development, but the functionality will remain the same, so please start there, and we can tweak the UI after I return.

Desktop mode:
I suggest starting with the button graphic from Basic Viewer. This button could sit in line with the navigation widgets on the left side of the screen. Clicking the widget would expand a panel (like the search does) containing the legend content. This panel should be styled to fit in with Reporter. Do not use the same styling from Basic Viewer.

Smartphone mode:
Let's try adding the item 'Legend' to the burger menu list in between Sign In and Help. Clicking this should open a popover window (like the help) with the legend info.

@allisonmuise allisonmuise removed the Low label Aug 15, 2017
@CTLocalGovTeam
Copy link
Contributor

@allisonmuise

Here is the screenshot of legend widget in desktop and mobile mode.

image

Please confirm if this looks good.

@allisonmuise
Copy link
Contributor Author

@CTLocalGovTeam I like it!

Will the access point for the legend be the same on desktop and mobile, or is it under the burger?

cc @ScottOppmann

@CTLocalGovTeam
Copy link
Contributor

@allisonmuise

We are planning to keep the access points same for mobile and desktop. This way user will always see Legend button irrespective of mobile/desktop mode.

@allisonmuise
Copy link
Contributor Author

@CTLocalGovTeam : Sounds good! Thanks!

CTLocalGovTeam added a commit to CTLocalGovTeam/crowdsource-reporter that referenced this issue Oct 30, 2017
@CTLocalGovTeam
Crowdsource Reporter AGOL 5.4 Enhancements (Sprint 1/2)
230760d 
Resolved following github tickets :
• Esri#226 : add basemap widget
• Esri#229 : add legend widget
• Esri#265 : Option to configure which geocoders are used in search
• Esri#299 : Cant submit new report when map JSON doesn't include popup
configuration
• Esri#301 : One map with multiple layers
• Esri#311 : BUG-000106163 Using the Citizen Problem Reporter template in
ArcGIS Online, if a custom geocoding service is used, and a point is
clicked on the map, the address of the location does not get populated
in the specified field.
@CTLocalGovTeam CTLocalGovTeam mentioned this issue Oct 30, 2017
Merged
@allisonmuise
Copy link
Contributor Author

@CTLocalGovTeam Could we make this configurable, and off by default?

@CTLocalGovTeam
Copy link
Contributor

@allisonmuise

We can add a new key showLegend in the config file (default value will be false) and show it in configuration panel.

@allisonmuise
Copy link
Contributor Author

@CTLocalGovTeam Thank you!! I realize I never wrote this into the issue description, so I appreciate you adding this parameter.

CTLocalGovTeam added a commit to CTLocalGovTeam/crowdsource-reporter that referenced this issue Nov 3, 2017
@CTLocalGovTeam
Crowdsource Reporter AGOL 5.4 Enhancements (Sprint 2/2)
e19ff31 
Resolved following github tickets :
• Esri#226 : add basemap widget
• Esri#229 : add legend widget
• Esri#310 : App doesn't display features symbolized using an Arcade
expression
• Esri#326 : CSS issue when report has no title
@CTLocalGovTeam CTLocalGovTeam mentioned this issue Nov 3, 2017
Merged
@allisonmuise
Copy link
Contributor Author

verified

@skrishn skrishn removed the test case label Nov 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CTLocalGovTeam CTLocalGovTeam

Labels
Enhancement High Verified
Projects
None yet
Milestone
AGOL 5.4
Development

No branches or pull requests
3 participants
@allisonmuise @CTLocalGovTeam @skrishn