-
Notifications
You must be signed in to change notification settings - Fork 78
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
Unearthing PVD map layers planning issue #448
Comments
A quick update: i was able to get the bounding customization working by including this script into the page: // find map object and limit bounds/zoom
window[Object.keys(window).find(key => key.substr(0,3) === "map")]
.setMaxBounds([
[42.2102, -72.0204],
[41.2272, -70.9618]
]).setMinZoom(9) It's working now at https://publiclab.org/wiki/unearthing-pvd |
Hi @tmarlow - i'd like to make some progress on the marker styling issue. My understanding was that the larger black markers seem to "overwhelm" the smaller blue markers. I'm cautious about making the blue markers larger because they are so crowded already. Could one possible solution be to override the marker display for this page and make the black markers smaller? Do you prefer another set of colors? Thanks! |
Hey Jeff,
Sorry, I don't think I was seeing those earlier messages!
The max bounds solution is perfect.
I think making them larger is not the way to go. Your solution makes more
sense to me. I don't really have a preference on the colors. I trust your
judgement more than mine on that one.
I've been swamped with other things but I have time the rest of this week
to make some progress. I'll keep an eye out and try to be more responsive
as well. Sorry, and thanks for your hard work!
…-Tom
On Tue, Apr 7, 2020 at 11:59 AM Jeffrey Warren ***@***.***> wrote:
Hi @tmarlow <https://github.com/tmarlow> - i'd like to make some progress
on the marker styling issue. My understanding was that the larger black
markers seem to "overwhelm" the smaller blue markers.
I'm cautious about making the blue markers larger because they are so
crowded already.
Could one possible solution be to override the marker display for this
page and make the black markers smaller? Do you prefer another set of
colors? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#448 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD65ZLAL3N5TS3OKMMP2HR3RLNEXPANCNFSM4LS2FKPA>
.
--
Thomas Marlow (he/him)
Brown University
Department of Sociology, Ph.D. Candidate
Trainee, Brown Superfund Research Program
Graduate Fellow, Institute at Brown for Environment and Society
|
No problem, and a lot of small but important things (like search) are coming together finally and i hope will go live today. Good luck with your work and stay safe! |
Search is now live on https://publiclab.org/map ! And on the unearthing pvd page! https://publiclab.org/wiki/unearthing-pvd |
@tmarlow - hi, just touching base about the map layers you're looking to add. cc @emilyashley 👋
My list includes:
map.setMaxBounds()
(see below)Icon size discussion
If the above problem statement is right, we could:
Cloning layers
The original layer we're looking at is here: https://github.com/publiclab/leaflet-environmental-layers/blob/master/src/unearthing.js so we'll copy it as many times as we need.
Change checklist:
unearthing-laundromats.js
However, after copying and modifying that file, we also need to change these files and lines to include the new layer:
leaflet-environmental-layers/src/info.json
Lines 277 to 296 in bfe2008
For the
info.json
file above, we need these fields updated for the new layers; if you can add that in a comment here:Once you do the above steps, please open a pull request with your work! Or, if you'd like, point us (in a comment here) to your branch and we can open it for you. Thanks Tom!!!
We can break this out into different issues as needed. Thanks!
The text was updated successfully, but these errors were encountered: