Companies' map on phones: the Table link as its icon, with a tooltip #1016
No reviewers
Labels
No labels
accessibility
authentication
breaking change
bug
documentation
enhancement
interface
internationalisation
observability
performance
security
tier
1
tier
2
tier
3
tier/4
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
Postulo/postulo!1016
Loading…
Reference in a new issue
No description provided.
Delete branch "916-map-table-icon"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #916
Builds on the #915 PR (#1013) and must be merged after it. The branch is cut from
915-map-tooltips-beside, so until #1013 is merged this PR's diff also shows that commit; it needs thedata-tooltip-placehook.On a phone (below
sm) the Table link on the companies' map is its icon alone, with a "Table view" tooltip below it.company_map.html: the word is<span class="sr-only sm:not-sr-only">Table</span>(visually hidden, not removed, so the name stays "Table"); the link is a square icon button belowsm; the tooltip isaria-describedby, asks forbelow, and is markeddata-tooltip-narrow.app.js:placeTipdoes not show adata-tooltip-narrowtooltip fromsm(40rem) up, where the word is visible; the server writes ithidden, which the script lifts when it makes it a tooltip.tests/e2e/test_map.py(390px: icon-only, name "Table", tip below on hover and focus; 1280px: word shown, no tip),tests/test_company_map.py(markup). Full unit suite green.Library check: none needed, a small UI change.
Model use: written with Claude Code, reviewed by the person who opens this PR.
🤖 Generated with Claude Code
a9629a652ca5f3301ee8