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.

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

undeclusterable opens side list #18

Closed
amitay7 opened this issue Jan 10, 2016 · 6 comments
Closed

undeclusterable opens side list #18

amitay7 opened this issue Jan 10, 2016 · 6 comments
Assignees
Milestone

Comments

@amitay7
Copy link
Collaborator

amitay7 commented Jan 10, 2016

No description provided.

@amitay7
Copy link
Collaborator Author

amitay7 commented Jan 20, 2016

suggested designs
image

image

@mushon
Copy link
Contributor

mushon commented Jan 22, 2016

Cool.
I think option2 works better than option1, as we don't use that diagonal line anywhere else.

  • Please try to check a darker color for the metadata (research name + year) and for the tags. Right now it looks slightly too pale.
  • Please add the time-frame to the title of the list.
  • we need a hover state for the list
  • does the panel open on hover or on click?

@yuvadm
Copy link
Member

yuvadm commented Jan 24, 2016

@mushon heads up that there are some limitations on which elements we can use as markers on the map

mapbox/mapbox-gl-js#358

@mushon mushon assigned amitay7 and unassigned yuvadm Feb 2, 2016
@mushon
Copy link
Contributor

mushon commented Feb 2, 2016

functionality implemented, please review design feedback above ^

@mushon mushon added this to the Version 0.7 milestone Feb 24, 2016
@mushon
Copy link
Contributor

mushon commented Feb 25, 2016

@amitay7 this should be addressed with #17 and closed when #17 is done

@amitay7
Copy link
Collaborator Author

amitay7 commented Mar 14, 2016

uploads/45197222-6c17-48e5-a078-ee21cee4318c/blob

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants