Skip to content

Conversation

@maparent
Copy link
Collaborator

@linear
Copy link

linear bot commented Aug 20, 2025

@supabase
Copy link

supabase bot commented Aug 20, 2025

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@maparent maparent requested a review from jsmorabito August 20, 2025 01:23
@maparent maparent force-pushed the eng-623-implement-new-button-hover-states-for-buttons-in-user branch from 46f8cc2 to de01d7e Compare August 24, 2025 17:29
@maparent
Copy link
Collaborator Author

This is what it looks like now. Hover on first button. Low contrast imho
https://loom.com/i/7dbff49376884bca840c88d1b4703fd3

Copy link
Collaborator

@jsmorabito jsmorabito left a comment

Choose a reason for hiding this comment

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

re hover color I'm following Roam's design for overall consistency. I agree its a little low contrast, but this will tide us over until a more thorough revamp of styles is warranted

@maparent maparent merged commit 9c37c55 into main Aug 27, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this to Done in General Aug 27, 2025
@maparent maparent deleted the eng-623-implement-new-button-hover-states-for-buttons-in-user branch August 27, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants