Implement new route-display design #743

KevinEtchells · 2024-07-09T07:28:20Z

Context

Update the route display to match the latest designs. This is the first step to making this visible to all users.

Check the route displays okay
That the tooltip appears on mouse-over, and disappears on mouse-leave is removed
This also works on focus (for accessibility) and click (for touch-screens)

brunns

Shouldn't show route for user messages.

Also, for AI messages, we shouldn't show the route before we've received one.

brunns

Belay that - had to hard-refresh my browser.

KevinEtchells added 3 commits July 9, 2024 08:21

Implement new route-display design

1d77078

Update integration tests

b71fa8d

Fix tooltip role accessibility issue

4bfb84e

KevinEtchells temporarily deployed to release July 9, 2024 08:00 — with GitHub Actions Inactive

KevinEtchells had a problem deploying to release July 9, 2024 08:00 — with GitHub Actions Failure

Hide route until ready

ea6e3e9

KevinEtchells had a problem deploying to release July 9, 2024 08:54 — with GitHub Actions Failure

KevinEtchells temporarily deployed to release July 9, 2024 08:54 — with GitHub Actions Inactive

Remove test_user_cannot_see_route test

71bb9d6

KevinEtchells temporarily deployed to release July 9, 2024 09:17 — with GitHub Actions Inactive

brunns suggested changes Jul 9, 2024

View reviewed changes

brunns approved these changes Jul 9, 2024

View reviewed changes

KevinEtchells merged commit f9e26bb into main Jul 9, 2024
9 checks passed

KevinEtchells deleted the feature/route-display branch July 9, 2024 09:59