fix(desktop): cap share popover port list height and scroll

Merged
HA
3d ago
+1-1

Summary

  • The "Ports on this computer" list in the Share workspace popover now stops at 240px (about 8 rows) and scrolls inside itself. Before, a workspace with many detected ports (for example, around 23 workerd/electron/MainThread ports) made the popover grow past the window.

Notes

  • This sets a max height, not a fixed height, so a short list doesn't leave empty space under it. The header, invite box and access sections stay where they are.

Testing

  • bunx biome check on the changed file
  • Not checked in the running app

Summary by cubic

Caps the "Ports on this computer" list in the Share workspace popover at 240px (about 8 rows) and makes it scroll inside itself. Before, a workspace with many detected ports made the popover grow past the window.

Uses a max height rather than a fixed one, so a short list doesn't leave empty space under it. The header, invite box, and access sections stay where they are.

Summary by CodeRabbit

  • Style
    • Long port lists now scroll within a height-limited container.