Open jywarren opened 4 years ago
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 notifications@github.com 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 https://github.com/publiclab/leaflet-environmental-layers/issues/448#issuecomment-610472390, 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:
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!