Skip to content

Conversation

@jaaydenh
Copy link
Contributor

@jaaydenh jaaydenh commented Dec 11, 2025

Screenshot 2025-12-11 at 13 10 01

@jaaydenh jaaydenh self-assigned this Dec 11, 2025
@jaaydenh jaaydenh requested a review from aslilac December 11, 2025 13:34
Copy link
Member

@aslilac aslilac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good, but design wise...

  • in the original design, the settings link was a button
  • I feel like things look a little vertically cramped. could we add just a bit more gap/padding in between?
Image

@geokat geokat force-pushed the geokat/internal-858-sharing-info-in-workspaces-response branch from 629e5b4 to 73a619c Compare December 12, 2025 15:16
@jaaydenh
Copy link
Contributor Author

@aslilac I was referencing the new designs created by christin, https://www.figma.com/design/CNr6qGcPhbq3Qut6rENz6i/Shared-Workspaces?node-id=168-3958&t=Ga6q3znU5CzyVLZO-1

Good catch on the settings button, I changed this because of the designs but I actually think the button is the correct UI here.

Screenshot 2025-12-12 at 15 10 55

@geokat geokat force-pushed the geokat/internal-858-sharing-info-in-workspaces-response branch from 73a619c to a50b597 Compare December 12, 2025 16:37
Base automatically changed from geokat/internal-858-sharing-info-in-workspaces-response to main December 15, 2025 16:42
@jaaydenh jaaydenh force-pushed the jaaydenh/sharing-indicator branch from 162f4b8 to 8d4ccbf Compare December 15, 2025 16:52
@jaaydenh jaaydenh merged commit 547e53f into main Dec 15, 2025
48 of 50 checks passed
@jaaydenh jaaydenh deleted the jaaydenh/sharing-indicator branch December 15, 2025 20:20
@github-actions github-actions bot locked and limited conversation to collaborators Dec 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants