-
Notifications
You must be signed in to change notification settings - Fork 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.
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
Map pin popups need more data shown #584
Comments
@andreadelrio
*Maybe add summary (if we can get around the “PROBLEMS AND PURPOSE” header from clipped body when summary is empty), related to #659 |
|
@jesicarson I think putting a snippet of the body is going to be tricky because we have a lot of different common headers we'd need to remove. Also, it'd be a crowded popup, so I'd advice against it. |
Thanks for this. Looking good so far! I hear you about including the short description... I think we can be ok with leaving that out (let's ask Amber). I see some typographic hierarchy and spacing issues with the layout of the popup text - can you compare to the mockup and see if you can improve? (Match to the layout of the CASE Title and Submitted by text in the grid below) I also think we should find a different location for the "this matched your search" text, as it is not part of the data. Perhaps the very bottom, aligned with the X? Thanks!! |
Jesi will provide mockups.
Not required before user testing.
The text was updated successfully, but these errors were encountered: