ncsco / pinemap-dss-help

Issue tracker for PINEMAP DSS
0 stars 0 forks source link

Seedling deployment tool legend updates #49

Closed daviswx closed 9 years ago

daviswx commented 9 years ago

When viewing the 5° ranges, show the rounded temperatures in the attached legends and add the map marker icon below the isotherm temperature.

daviswx commented 9 years ago

This is done, but IMO having the marker icon hanging out in the legend begs the question of why it's there. So we may need to add a tooltip somewhere in the legend explaining what these ranges represent and what temperature they're centered on.

I will brainstorm some ideas, and maybe we can discuss this at our "pow-wow" tomorrow.

hadinon commented 9 years ago

Yes, I agree. Let's discuss at our "pow-wow"!

daviswx commented 9 years ago
hadinon commented 9 years ago

Super minor detail: I just noticed that we've been calling the historical maps "average annual minimum temperature" on the seedling deployment tools so we may want to adjust the Seedling Markets tooltip wording to match that.

daviswx commented 9 years ago

These new tooltips are in place. I'm not sure how good the icon looks positioned in the top right corner of the marker. I also tried putting it at the bottom right but it didn't necessarily look any better. Any suggestions?

hadinon commented 9 years ago

Hmmm what if you change it to look something like this (see below center map)? image I "inspected the element" in Chrome and changed the tooltip style= to say "left: 2px;".

daviswx commented 9 years ago

Sure, I think that looks a bit better. I changed it and set it to 1px.

One other small thing I just discovered with the seedling deployment tools is that the marker labels don't show a ".0" for temperatures that round evenly to an integer. I will fix that to be consistent with the other tools.

hadinon commented 9 years ago

Another idea: is it possible to put the tooltip to the right of the bolded number? Here's what that looks like but it seems a bit cluttered to me.

image

daviswx commented 9 years ago

I thought about that as well, but like you said, I think it's just too cluttered and it may not even be clear which number it applies to.

Also, the number rounding issue should be fixed, so all numbers will show one decimal place on the map labels and in the attached legends.

hadinon commented 9 years ago

Agreed. In the tooltip, do you think we should mention something about how the 5 degree ranges represent 1° cooler to 5° warmer than that min/mean/max projected temperature?

daviswx commented 9 years ago

I would say no for those tooltips since they specifically apply to that one temperature value. And hopefully that part is self-explanatory in the legend since the temperatures are listed there.

hadinon commented 9 years ago

Yeah, that's true. OK I'm going to go ahead and close this issue.