311-data icon indicating copy to clipboard operation
311-data copied to clipboard

Display NC names on hover via pop-up modal

Open cottonchristopher opened this issue 1 year ago • 10 comments

Overview

We need to display NC names on-hover over unselected NC, so that users can have the necessary information to increase functionality and ease of use ( knowing NC names)

Action Items

  1. on-hover (over un-selected NC), display pop-up modal with NC name
  • [ ] Address the problem overview with possible design solution(s) in a comment
  • [ ] Complete Design Iterations section below
  • [ ] Document user interaction in Figma

Resources

v1.2 Moderated Usability Study


Design Iterations

Please move ticket between In Progress and In Review to assist PM team

Iteration 1

Link to notes: REPLACE WITH COMMENT URL

REPLACE WITH SCREENSHOT UPLOAD


Hand Off Materials

Figma Section Name: REPLACE WITH SECTION NAME

Before Screenshot

<image>

After Screenshot (Finalized)

REPLACE WITH SCREENSHOT UPLOAD

Designer Resources

Iteration Dropdown Copy/Paste

<details><summary>Iteration X</summary>
<p>

Link to notes: `REPLACE WITH COMMENT URL`

`REPLACE WITH SCREENSHOT UPLOAD`

</p>
</details> 
Instructions for Engineering Hand Off

To Start Engineering Hand Off...

  1. Ensure all Hand Off Materials are filled in
  2. Add the "ready for dev lead" label
  3. Leave a comment saying "This ticket is ready for engineering hand off."

cottonchristopher avatar Sep 01 '24 23:09 cottonchristopher

Merging this ticket with: https://github.com/hackforla/311-data/issues/1808

cottonchristopher avatar Sep 06 '24 03:09 cottonchristopher

Ready for Design Lead, please add the appropriate size level label based on review

cottonchristopher avatar Sep 06 '24 03:09 cottonchristopher

Added some links to usability test presentations, referencing users that are asking for this (or similar) feature

ryanfchase avatar Sep 07 '24 08:09 ryanfchase

Inspiration Research: Weather.com, LA city, California

Screenshot

Image

Iteration 1:

NC Label

Image

Hover Popup

Image

Joy-Truex avatar Sep 25 '24 07:09 Joy-Truex

@Joy-Truex Thanks for your work on this. Should we include your screenshots as "Iteration 1" in the original post?

Also, a tip on pasting images in dropdowns, the ![Image](...) line needs to have an empty line above and below it. For some reason the image doesn't know it should render unless it's got some buffer lines.

Example:

<details><summary>Details</summary>
<p>

![Image](...) <-- an empty line above and below

</p>
</details> 

ryanfchase avatar Sep 25 '24 21:09 ryanfchase

@ryanfchase Do you know if this ticket is in development? This will impact my work on the hover popup. Thanks!

Joy-Truex avatar Sep 26 '24 03:09 Joy-Truex

@ryanfchase Do you know if this ticket is in development? This will impact my work on the hover popup. Thanks!

The link goes to a figma section for #1699, is that the ticket you refer to?

If so, the work is completed and the ticket is closed:

  • https://github.com/hackforla/311-data/issues/1699

ryanfchase avatar Sep 26 '24 05:09 ryanfchase

@ryanfchase Do you know if this ticket is in development? This will impact my work on the hover popup. Thanks!

The link goes to a figma section for #1699, is that the ticket you refer to?

If so, the work is completed and the ticket is closed:

No unfortunately not! the ticket i'm referencing is the content of that frame, not the label which seems to be incorrect, but I cant find the actual correct ticket

can you let me know if we have something in development to change to what is in the frame? I was asked to work on this (independently of a ticket??) when i first became a lead so i cant find or recall the exact trail...:(

Joy-Truex avatar Oct 30 '24 04:10 Joy-Truex

Iteration 1.1

After some exploration, I have come up with this option for showing the NC label upon hovering over a NC.

View 1

Zoomed out and hovering over a not-selected NC (in this case Pico Union) Screenshot 2024-10-29 at 9 33 42 PM

View 2

Zoomed in and hovering over a not-selected NC (in this case Wilshire Center) Screenshot 2024-10-29 at 9 32 55 PM

Joy-Truex avatar Oct 30 '24 04:10 Joy-Truex

Iteration 2.1

After some exploration, I have come up with this option for showing the NC label upon hovering over a 311 ticket.

please note that this shows the location labeled as opposed to previously when it was unclear to some users what NC a ticket was in Screenshot 2024-10-29 at 9 43 34 PM

Screenshot 2024-10-29 at 9 43 40 PM

Joy-Truex avatar Oct 30 '24 04:10 Joy-Truex

@ryanfchase Do you know if this ticket is in development? This will impact my work on the hover popup. Thanks!

The link goes to a figma section for #1699, is that the ticket you refer to? If so, the work is completed and the ticket is closed:

No unfortunately not! the ticket i'm referencing is the content of that frame, not the label which seems to be incorrect, but I cant find the actual correct ticket

can you let me know if we have something in development to change to what is in the frame? I was asked to work on this (independently of a ticket??) when i first became a lead so i cant find or recall the exact trail...:(

FINALLY FOUND THE TICKET!! 1669 Any idea what became of this? I'm having trouble following the trail as to if it ever went to dev and why? @ryanfchase

Joy-Truex avatar Nov 14 '24 02:11 Joy-Truex

Here is where we can find the specifications for that design update Screenshot 2024-11-20 at 7 09 09 PM

Joy-Truex avatar Nov 21 '24 03:11 Joy-Truex

Here is where we can find the specifications for that design update

A dev ticket has been opened to implement this design: https://github.com/hackforla/311-data/issues/1870, I've left a question for design leads.

@Joy-Truex let us know the status of this ticket. Does it need review, or is it ready to be closed and ready for dev? I'll add this to the upcoming agenda to make sure your work on the SR details design is properly wrapped up before it is implemented. Thanks!

Edit: updated my questions several times, lol

ryanfchase avatar Jan 03 '25 09:01 ryanfchase

@ryanfchase the implementation of #1870 solves for the issue when there is a SR to hover over, however not when the user is trying to identify a NC independently of a SR

I also have created a design for this second instance but I want your opinion on whether this is out of scope for this ticket.

Design

Image

Joy-Truex avatar Jan 26 '25 19:01 Joy-Truex

@Joy-Truex I think it is within scope? see "Show NC names while users hover over the NC district area on the map" from our 2024 usability test presentation.

But let's put this on tonight's agenda just to confirm

ryanfchase avatar Jan 29 '25 20:01 ryanfchase

if confirmed in scope please note need additional information from dev team about how to confirm what zoom % the label appears at!

Joy-Truex avatar Jan 29 '25 23:01 Joy-Truex

Hey @Joy-Truex !

Just to clarify before our meeting, is the current ask from DEV the following?

  • Provide zoom % for when the 2 types of popups appears.

Type of Popups:

  • Show NC name on hover when zoomed out.
  • Show NC name on hover when zoomed in.

traycn avatar Jan 30 '25 02:01 traycn

yes + how i would indicate to the team what level i want the NC popup to appear at as it doesnt make sense to show up when its all the way zoomed out and only should start to show up at a certain point.

Joy-Truex avatar Jan 30 '25 02:01 Joy-Truex

@ryanfchase this seems to be trapped in questions. Can you please review? To remind you of where we left off: https://github.com/hackforla/311-data/issues/1807#issuecomment-2623391927

the design is basically completed just need to know how to specify for the dev handoff. Thank you!

Joy-Truex avatar Feb 20 '25 01:02 Joy-Truex

ETA for answering this will be 2/23, so sorry for the delay -- needing extra time this week.

ryanfchase avatar Feb 20 '25 05:02 ryanfchase

@Joy-Truex speaking from a dev perspective, specifying different hover behaviors based on zoom level adds a large degree of complexity. It is technically feasible, but we now must internally maintain "states" of the map that the user is not aware of. Also, most map applications with popup-labels will concede that there will be a level of zoom where the labels can block other map content. Maps with clustered data points are the only visualizations (that I know of) that "get out of the way" depending on zoom level.

A couple of alternatives if that doesn't work for you:

  • we can allow users to toggle NC names on hover, e.g. as part of a setting. I know this would be the first type of control that is a "user setting", but this lets us provide the on-hover labels and also lets the user get rid of it if they don't find it useful.
  • if the user has not selected an NC already, hovering over an NC can populate the Neighborhood Councils dropdown, perhaps with reduced opacity to suggest that the user can select this NC

ryanfchase avatar Mar 01 '25 02:03 ryanfchase

So am I correct in assuming that the popup hover label would appear at any zoom level?

Mockup Screenshot

Zoom Out, Zoom In

Image

Image

Joy-Truex avatar Mar 03 '25 18:03 Joy-Truex

As the resident data visualization developer, yes, that is typically the behavior for labels on-hover. They will appear at any zoom level.

ryanfchase avatar Mar 03 '25 20:03 ryanfchase

Thank you! I have asked for some quick feedback from the design team, is it alright if I give a final response after GM Wednesday? Feel free to assign back to me and thumbs up if so :)

Joy-Truex avatar Mar 03 '25 22:03 Joy-Truex

No problem, let's do this instead. This can stay in Questions, and I'll add a couple labels to indicate that its awaiting team discussion -- and I'll keep ready for design lead so that we know its not needing input from another team

ryanfchase avatar Mar 03 '25 22:03 ryanfchase

Regarding display NC names on-hover over unselected NC:

After discussing with the design team, I have determined that it makes sense for the NC label to appear at any zoom level.

It may be a bit clunky to use at a fully zoomed out perspective, however this is partially addressed by the max zoom out being preset & the fact that users can zoom in more in order to be more precise as needed. Benefits include consistency with other map UI's and scalability/ease of use on a larger scale use for users who are looking for a less-precise view.

Joy-Truex avatar Mar 20 '25 02:03 Joy-Truex

Ryan (dev lead) to make a ticket where we QA the live version of the modal with the design to ensure that:

  • [ ] the modal matches the design
  • [ ] the modal meets requirements

Joy-Truex avatar Mar 20 '25 02:03 Joy-Truex

Once this has gone through QA ticket (to be made following return to 311Data from TDM hiatus) it will be prepared for handoff.

Joy-Truex avatar Apr 03 '25 18:04 Joy-Truex

@Joy-Truex I'm needing help getting a recap on this ticket. So, it looks like the work completed for this ticket is completed (you provided screenshots here and here). And we decided that this hover modal should appear at any zoom level (citation here, where you mention tradeoffs and benefits). Is that all correct? Did you have a link to your work in Figma?

Finally, I am happy to move forward on a QA ticket for the NC Details Modal (we even made note of it at the previous GM). Can you explain why it blocks the completion of your work on this ticket? Perhaps I missed something, but I am not seeing a reason why we cannot hand this off to dev and do the QA work simultaneously.

NC Details Modal

Screenshot 2024-10-29 at 9 43 34 PM

ryanfchase avatar May 09 '25 05:05 ryanfchase

Based on a Slack Huddle with PM and Design Lead, we agree that there are two concerns addressed by this ticket:

  1. Hovering over a service request on the map, which results in SR Details Modal displaying the NC name
  2. Displaying NC names when hovering over unselected NC regions on the map, which will result in that NC name appearing in a modal.

The designs for the SR Details modal was completed in previous tickets (design + dev), but Design Lead notes that there are still discrepancies remaining between the design specification and the current implementation. We have decided to create a new ticket for the design team to list all of these discrepancies, and another dev ticket will be created to fix them.

  • design follow up ticket: https://github.com/hackforla/311-data/issues/1979
  • dev follow up ticket: https://github.com/hackforla/311-data/issues/1980

The designs for NC Name-Preview modal are present within this ticket. The designs are completed, and the work is ready for dev hand off.

  • dev follow up ticket: https://github.com/hackforla/311-data/issues/1981

ryanfchase avatar May 26 '25 00:05 ryanfchase