project-lux / lux-frontend

Web front end of LUX
Apache License 2.0
3 stars 0 forks source link

Add on hover to entity type icons on results pages, entity pages, advanced search (any place icons are used) (from 1607) #22

Closed gigamorph closed 1 week ago

gigamorph commented 8 months ago

Problem description: The icons displayed alongside the results and in the entity page headers do not have visible labels displayed.

Expected Behavior: When hovering over an icon, the user should be shown text that describes the icon. There should be no hover delay. The label will be visible/invisible to the user immediately when their cursor enters/leaves the icon area. This should be implemented wherever icons are used in the UI.

Requirements

Needed for promotion

image

UAT Current Labels and link for UAT

  1. Scores and Notation
  2. Photographs
  3. Furniture
  4. Journals & Newspapers
  5. Clothing
  6. Numismatics
  7. Cartography
  8. Video
  9. Audio
  10. Software & Electronic Media
  11. Visual Works
  12. Textual Works
  13. Objects
  14. Specimens
  15. Collections

Dependencies/Blocks

Related Issues

Mockup/Wireframes:

image
roamye commented 1 month ago

This is no longer blocked. Moving this to backlog and assigning @kamerynB.

note to self: update links and reformat issue.

roamye commented 3 weeks ago

inconsistent with:

things to note: Under works, if a work is a digital work, it will show "digital work" when hovering over it rather than (example) Under works, textual object icons also display "linguistic object". example Set icon will appear as 'set and collection" if applicable. I think this is good that it can differentiate it, but wanted to note it.

Unsure if these are issues.

kkdavis14 commented 3 weeks ago

I think we should swap the "linguistic object" hover over to "textual work" since that's what the UI uses generally.

roamye commented 3 weeks ago

uat 10/24: hot fixes:

Quality control is needed for all the labels.

Next steps:

clarkepeterf commented 3 weeks ago

This is not in PRD, so I will deploy to lower envs

roamye commented 2 weeks ago

Approved by UAT, but as mentioned in the comment above, this will need another ticket for re-labeling.

roamye commented 1 week ago

Looks good in prod. closing