ncsco / pinemap-dss-help

Issue tracker for PINEMAP DSS
0 stars 0 forks source link

Three-map Layout #31

Closed hadinon closed 8 years ago

hadinon commented 8 years ago

Change occurrence map titles to “Minimum Projected Occurrence”, “Mean Projected Occurrence”, “Maximum Projected Occurrence” and add tooltips to explain these (e.g. minimum projected occurrence = historical average + minimum projected change) On the enlarged map title, disable the hover-over mouse pointer Disable clicking on the small maps to select a location, clicking on them should enlarge them Add a loading icon over the map(s) while layers are loading (HIGH PRIORITY)

daviswx commented 8 years ago

A few updates and questions about this:

daviswx commented 8 years ago

More updates:

For now, I will leave this issue open but flag it as a question since we will wait to hear from the beta testers about the small map interaction.

hadinon commented 8 years ago

Below is some feedback regarding the last item in this issue... the need for a loading icon. One of our testers, Steven Weaver, says:

In Chrome browser, tested all tools, and lag noted:

For loading issues, in summary:

Note:

hadinon commented 8 years ago

A few comments...

daviswx commented 8 years ago

Regarding the loading icon, I have some good news and some bad news.

I should qualify that with one caveat: Apparently they have added some experimental tile load events in the latest version, 3.7.0. The DSS is running v3.0.0, though, and upgrading isn't as simple as changing the version number. As versions have changed, so has the syntax for a few important features, so upgrading would require re-writing chunks of the map code.

I'm not really sure how long that would take, but since we're so close to the next round of beta testing, I'd be a bit afraid to start upgrading now if it took a while or unexpected bugs came up.

So here's what I'm thinking:

Sound like a good plan to you?

daviswx commented 8 years ago

One other change: As we discussed earlier, I have changed the Historical Occurrence and Projected Occurrence options to Historical Average and Projected Average. This change should be reflected in the menu options, the map titles, and the timeseries.

hadinon commented 8 years ago

I know we just discussed this but I'll write it here just in case we ever need to look back at this!

I'm good with all the above changes and agree with your proposed plan for moving forward. I've updated the beta tester question document to reflect the items/questions mentioned above.

Lastly, I agree with what you mentioned during our discussion -- because the loading icon is tied to location information right now, let's stick with the "Loading Location Data" on-screen text.

hadinon commented 8 years ago

Steven Weaver noticed a significant lag in loading the seedling deployment tools the other day (mentioned above). He just confirmed that this significant lag is no longer present.

daviswx commented 8 years ago

After updating the maps to OpenLayers v3.4.0, I was able to add a progress bar when the map layers are loading. It seems to be working for all tools on the development site.

I think that's the only outstanding topic with this issue, so I am closing it. Please re-open (or start a new issue) if problems are found with the loading bar.

hadinon commented 8 years ago

We still need to resolve the small map interaction, right? If that is already listed in another issue please feel free to re-close this. During the small group beta tester calls, I believe we plan to ask their opinion on enlarging the small map AND dropping a marker upon clicking the map. I'm guessing they will be OK with it. Once that's implemented, this should be good to close.

hadinon commented 8 years ago

Action items from beta testing: Change wording to “Highest/Lowest Likely” (per IPCC likelihood guidance -- https://www.ipcc.ch/pdf/supporting-material/uncertainty-guidance-note.pdf) and “Multi-model Mean”. Add subtitles saying “2.5th Percentile” and “97.5th Percentile”. Keep tooltips in map titles for change and projected average displays stating that these represent two std deviations above/below mean and two std deviations above/below mean + historical avg, respectively. Also clarify (with images) the percentiles and standard deviations in the FAQ box -- be sure to mention that this is an absolute change. Adjust the wording on the time series legend to say “range of likely possibilities/outcomes”. Also change the tooltip wording for the time series legend to note 2 standard deviations/percentiles. (HIGH PRIORITY)

hadinon commented 8 years ago

Additional action items from beta testing: Move the enlarge icon closer to the title text and add "ENLARGE" text beneath it. Allow dropping a marker on the side maps. *Add this to the FAQ for side maps: “you can zoom in on the side maps without changing the location that’s already selected” (MEDIUM PRIORITY)

hadinon commented 8 years ago

Action item from beta survey: *Make minor roads black (LOW PRIORITY)

daviswx commented 8 years ago

Here are two examples of "enlarge" button placement (with the updated icon):

Far right of title bar maps_enlarge_a

Centered with the title maps_enlarge_b

Any thoughts about these after seeing them on the page? When it's centered with the title (and tooltip), is it too cluttered? Or, like the beta testers suggested, do you think it's easier to see the icon this way?

Also, any thoughts on having the "ENLARGE" text below the icon?

hadinon commented 8 years ago

I really like it centered. How about you? The placement of "ENLARGE" below the icon looks great.

daviswx commented 8 years ago

I also like the "ENLARGE" text on it. As far as the placement goes, I don't have much of a preference one way or the other since there's only a small difference. But since you prefer the centered version, I went ahead and implemented that.

One very minor thing I've noticed is that when you hover over the side map titles, the underline extends one space to the right of the name and butts up against the Enlarge button. I wasn't able to find a quick solution to this after quickly playing around with the CSS, but I'm sure it can be fixed. If I have time, I will try to get back to it before the end of the week.

daviswx commented 8 years ago

Other updates for this issue:

Is that everything we needed to do regarding the maps?

hadinon commented 8 years ago

I see what you mean about the underline extending slightly and agree it is lower priority.

Yeah, good point about that side map note. I agree that is not necessary anymore.

Sounds good about the roads. Maybe we keep this open until that and the other minor underline issue is resolved?

Other than that, there were a few other action items listed above (from beta testing):

daviswx commented 8 years ago

A few updates about this so far:

hadinon commented 8 years ago

Yeah, I agree with "Highest/Lowest Likely Change" and "Highest/Lowest Likely Average" -- still don't 100% love that wording but it may be our best option at this point (I can't think of anything better).

As far as clarifying the percentiles and std deviations in the side map FAQ box, could it be as simple as re-wording the first sentence as, "...the multi-model mean, or the 2.5th percentile and 97.5th percentile"? I think defining this as an absolute change is covered in the Time Period tooltip. I'm thinking an image is not necessary. Thoughts?

Yep, I agree with your time series legend changes and not changing the time series tooltip.

hadinon commented 8 years ago

Do we also want to apply the "Highest/Lowest Likely" terminology and subtitles to the seedling deployment tools? Side maps might say, "Highest/Lowest Likely Temperatures" instead of "Minimum/Maximum Projected Temperatures". Center map title could say, "Multi-Model Mean Temperatures". Thoughts?

daviswx commented 8 years ago

Updated FAQ wording, map titles, and map subtitles are now in place.

daviswx commented 8 years ago

The updated updated map title wording ("Highest/Lowest Likely Change/Outcome/Temperatures") is now in place, and the subtitles are turned off.

hadinon commented 8 years ago

At our meeting yesterday, we talked about making the "Show Layers" box expand by default since some of the eye tracking participants couldn't seem to find it.

daviswx commented 8 years ago

I've tried something a bit different with the map controls to address this.

First, I expanded the Show Layer Options box, but it overlapped with most of VA and parts of NC within the native loblolly range. So I moved this box to the top left corner of the map and moved the zoom controls to the right side.

Take a look at it and let me know what you think.

hadinon commented 8 years ago

Yes, I love this! I'm going to close the issue (again, ha) but re-open if needed. :smile:

hadinon commented 8 years ago

One minor note: the first two paragraphs of the map FAQ box will need to be tweaked (switch "upper left corner" and "upper right corner"). I'm not going to re-open since it seems like an easy fix. :wink:

hadinon commented 8 years ago

Also, maybe we should just call it "Layer Options"? I don't vote for disabling the ability to click and hide it but just think it might be better to call it "Layer Options". Agree? Disagree?

daviswx commented 8 years ago

FAQ box wording is updated.

I'm fine with calling it "Layer Options". Maybe instead of saying "Show" or "Hide", we should just have a small :heavy_plus_sign: or :heavy_minus_sign: icon in the top right corner?

hadinon commented 8 years ago

Yes, I like the +/- idea. :+1:

daviswx commented 8 years ago

This is now in place.

hadinon commented 8 years ago

Minor tweak then we can close this: the map FAQ box needs to be changed to say, "Layer Options" instead of "Show Layer Options".

daviswx commented 8 years ago

Done. :+1:

hadinon commented 8 years ago

Alter text from "Loading Location Data" to add something like "Scroll to see chart below".

daviswx commented 8 years ago

I have updated the text to read "Loading location data on the map and in the time series below". Since the data loads fairly quickly, this message isn't on the screen long, so I wonder whether users will actually read it. Frankly, that is probably true with any text we'd put there.

My other idea with this was to add an overlay at the bottom of the screen when the time series loads. Once a user scrolls down, the overlay would disappear, but until then, it might be a useful cue that more information is available at the bottom of the page. Here's a mockup of how that might look:

timeseries_loadingoverlay

daviswx commented 8 years ago

I restored the map loading text to its original version ("Loading Location Data") and added an overlay at the bottom of the screen that appears when the map is clicked and disappears when the entire time series chart is visible.

daviswx commented 8 years ago

It sounds like we're all happy with this overlay at the bottom of the screen, and I don't think there were any other outstanding action items with this issue, so I am closing it.

daviswx commented 8 years ago

After viewing the DSS on large screens at the annual meeting, the time series loading notification at the bottom of the screen was less obvious than we hoped for. I will try changing the background color from the PINEMAP greenish-aqua color to a brighter green, like the one used when the location box flashes.

daviswx commented 8 years ago

I have changed the time series loading notification background to be a brighter green color. I also made it a bit taller and made the font slightly larger.

image

I think it should be more noticeable this way, but if we feel like it still isn't, I can make it pulse as well.

hadinon commented 8 years ago

Looks good to me. Closing this issue but feel free to re-open if necessary.