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

[events map] Can we make the z-index for future events > z-index for past events? #125

Closed
brianloveswords opened this issue Oct 29, 2014 · 6 comments

Comments

@brianloveswords
Copy link
Contributor

We have an event in NYC coming up but the yellow flag is buried under grey flags. Is there any way to get a consistently greater z-index on future events?

@sudodoki
Copy link
Contributor

Given new events are prepended to sheet and old events are on bottom, simple reverse() on data after response should do the trick. Let me experiment with this on my localhost.

@sudodoki
Copy link
Contributor

Actually, disregard my previous comment - there's some date sorting taking place.

@finnp
Copy link
Member

finnp commented Oct 29, 2014

Hm I also wasn't able to fix this. Seems like the styling of the marker can only be done though the attrebites of the simplestyle-spec, which doesn't support zIndex.

I am not sure what the best solution is. I think this would require a PR to sheetsee-maps, somehow supporting non-GeoJSON styling.

But maybe there is another way?

@jlord
Copy link
Contributor

jlord commented Oct 29, 2014

I just made the PR ☝️ which I think is the best fix. It sorts the dates before they go into the map (previously they were just being sorted before being listed, not before being mapped) and so now an upcoming NYC event will be above an older NYC event.

I don't think a z-index would work actually because you wouldn't want a NYC marker floating ontop of a Georgia marker.

@sudodoki
Copy link
Contributor

Is it a bad idea to suggest showing only active markers & have toggable option to see all?

@jlord
Copy link
Contributor

jlord commented Oct 29, 2014

@sudodoki I don't think it's a bad idea, I think there was just a want for a 💥 impact of showing all of them, but we could just have the front page show upcoming ones on the map and then the /events page can show all.

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

No branches or pull requests

4 participants