coracle/flotilla
1
6
Fork 23
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

feat: show space name on hover in primary nav (#129) #136

Merged
hodlbod merged 3 commits from priyanshu_bharti/flotilla:129-show-space-name-on-hover into dev 2026-04-06 16:08:03 +00:00
Conversation 4 Commits 3 Files Changed 7
+59 -52
priyanshu_bharti commented 2026-04-03 15:48:29 +00:00
Contributor
Copy Link
Copy Source

Resolves #129

This PR implements showing the human-readable space (relay) name when hovering over space icons in the primary navigation sidebar. It uses deriveRelayDisplay to ensure the name is correctly resolved from metadata.

  • Modified PrimaryNavItemSpace.svelte
  • Verified locally that the tooltip shows the expected name.
Resolves #129 This PR implements showing the human-readable space (relay) name when hovering over space icons in the primary navigation sidebar. It uses `deriveRelayDisplay` to ensure the name is correctly resolved from metadata. - Modified `PrimaryNavItemSpace.svelte` - Verified locally that the tooltip shows the expected name.
Screenshot from 2026-04-03 21-17-39.png
63 KiB
Screenshot from 2026-04-03 21-17-39.png
hodlbod commented 2026-04-03 18:28:04 +00:00
Owner
Copy Link
Copy Source

Interesting, it looks like this is a Firefox-only bug — in Zen browser the tooltip isn't appearing for me. In Chrome (Brave) it works fine. Could you test this in FF?

Interesting, it looks like this is a Firefox-only bug — in Zen browser the tooltip isn't appearing for me. In Chrome (Brave) it works fine. Could you test this in FF?
priyanshu_bharti commented 2026-04-03 22:32:47 +00:00
Author
Contributor
Copy Link
Copy Source

Hey @hodlbod,
Tested in Firefox (latest version) just now.
Tooltip is appearing correctly and showing the space name on hover in the primary nav ✅

Attached screenshot from Firefox for reference.
image.png

Working fine now in Firefox too!

Let me know if you want any other changes. Ready to merge whenever you are

Hey @hodlbod, Tested in **Firefox** (latest version) just now. Tooltip is appearing correctly and showing the space name on hover in the primary nav ✅ Attached screenshot from Firefox for reference. ![image.png](/attachments/78128bc3-e9a6-4faa-9bd9-47b6814e45f8) Working fine now in Firefox too! Let me know if you want any other changes. Ready to merge whenever you are
image.png
79 KiB
hodlbod commented 2026-04-04 14:35:10 +00:00
Owner
Copy Link
Copy Source

Thanks, this is getting there. A couple things:

  • Let's remove the changes to app.css. They look good, but I want to avoid one-off colors and !important for maintainability.
  • Use the cx utility instead of filter(Boolean).join
  • The tooltips still aren't showing up in FF on pages with a SecondaryNav — I think the nav is hiding the tooltips.
Thanks, this is getting there. A couple things: - [x] Let's remove the changes to app.css. They look good, but I want to avoid one-off colors and !important for maintainability. - [x] Use the `cx` utility instead of filter(Boolean).join - [x] The tooltips still aren't showing up in FF on pages with a SecondaryNav — I think the nav is hiding the tooltips.
priyanshu_bharti added 3 commits 2026-04-04 18:16:58 +00:00
feat: show space name on hover in primary nav (#129) 41fb6f95b2
feat: robust navigation tooltips for Firefox/Zen compatibility (#129) 503c2bbd40
style: Navigation tooltip improvements and Firefox fix 9ae3c8290a 
- Refactor PrimaryNavItem and SecondaryNavItem to use 'cx' utility.
- Add tooltip support to SecondaryNavItem and SpaceMenu room items.
- Remove hardcoded tooltip colors from app.css.
- Fix Firefox PrimaryNav tooltip visibility by increasing z-index and adding isolation: isolate.
priyanshu_bharti force-pushed 129-show-space-name-on-hover from 28f691753b to 9ae3c8290a 2026-04-04 18:16:58 +00:00 Compare
priyanshu_bharti commented 2026-04-04 18:21:09 +00:00
Author
Contributor
Copy Link
Copy Source

@hodlbod

All points addressed:

  • Reverted app.css changes
  • Switched to cx utility instead of filter(Boolean).join()
  • Fixed tooltip issue in Firefox on SecondaryNav pages (z-index + stacking context)

image.png

Also included space name hover improvement.

Ready for merge whenever you are. Thanks!

@hodlbod All points addressed: - Reverted app.css changes - Switched to `cx` utility instead of filter(Boolean).join() - Fixed tooltip issue in Firefox on SecondaryNav pages (z-index + stacking context) ![image.png](/attachments/aa084c04-871d-4d6f-8c82-0c78ce59b49c) Also included space name hover improvement. Ready for merge whenever you are. Thanks!
image.png
285 KiB
hodlbod merged commit fed34a2747 into dev 2026-04-06 16:08:03 +00:00
hodlbod deleted branch 129-show-space-name-on-hover 2026-04-06 16:08:03 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#136
Delete Branch "priyanshu_bharti/flotilla:129-show-space-name-on-hover"

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?