coracle/flotilla
1
6
Fork 24
Code Issues 38 Pull Requests 3 Actions Packages Projects 5 Releases Wiki Activity

fix(video): restyle spotlight pin button on video tiles #281

Merged
hodlbod merged 1 commits from userAdityaa/flotilla:pin-color into dev 2026-05-21 20:59:26 +00:00
Conversation 3 Commits 1 Files Changed 1
+2 -2
userAdityaa commented 2026-05-21 15:47:09 +00:00
Collaborator
Copy Link
Copy Source

Summary

  • Replace the semi-transparent purple spotlight pin (bg-primary/25) with a solid btn-primary button when a tile is pinned
  • Use an opaque btn-ghost bg-base-100 style for the unpinned state instead of bg-base-100/70

closes #267

### Summary - Replace the semi-transparent purple spotlight pin (`bg-primary/25`) with a solid `btn-primary` button when a tile is pinned - Use an opaque `btn-ghost bg-base-100` style for the unpinned state instead of `bg-base-100/70` closes #267
userAdityaa commented 2026-05-21 15:48:35 +00:00
Author
Collaborator
Copy Link
Copy Source

@hodlbod, thoughts on this ?
Should I keep a single toggle color scheme for both themes, with just selected and unselected states?

@hodlbod, thoughts on this ? Should I keep a single toggle color scheme for both themes, with just selected and unselected states?
hodlbod commented 2026-05-21 15:50:00 +00:00
Owner
Copy Link
Copy Source

I actually think we should stick with purple, it's much less unambiguous than white/black. The transparency is mostly what I was referring to as ugly. We should use a non-transparent purple button (which is just the standard btn btn-primary).

I actually think we should stick with purple, it's much less unambiguous than white/black. The transparency is mostly what I was referring to as ugly. We should use a non-transparent purple button (which is just the standard btn btn-primary).
👍 1
userAdityaa commented 2026-05-21 15:51:03 +00:00
Author
Collaborator
Copy Link
Copy Source

make sense, pushing the update shortly.

make sense, pushing the update shortly.
userAdityaa force-pushed pin-color from d3a25f4de6 to c1c00eca7e 2026-05-21 20:27:40 +00:00 Compare
userAdityaa marked the pull request as work in progress 2026-05-21 20:28:16 +00:00
userAdityaa added 1 commit 2026-05-21 20:33:00 +00:00
fix(video): use solid btn-primary for spotlight pin c2059b295c
userAdityaa force-pushed pin-color from c1c00eca7e to c2059b295c 2026-05-21 20:33:00 +00:00 Compare
userAdityaa marked the pull request as ready for review 2026-05-21 20:34:04 +00:00
hodlbod merged commit c441012e02 into dev 2026-05-21 20:59:26 +00:00
hodlbod deleted branch pin-color 2026-05-21 20:59:26 +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
AQB 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#281
Delete Branch "userAdityaa/flotilla:pin-color"

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?