Skip to content

Fix link in docs and add On<Release> interaction observer - #480

Merged
janhohenheim merged 3 commits into
TheBevyFlock:mainfrom
olekspickle:main
Feb 6, 2026
Merged

Fix link in docs and add On<Release> interaction observer#480
janhohenheim merged 3 commits into
TheBevyFlock:mainfrom
olekspickle:main

Conversation

@olekspickle

Copy link
Copy Markdown
Contributor

I found out I missed the release observer that resets color after button press

@benfrankel

Copy link
Copy Markdown
Collaborator

Wouldn't a pointer release change the bg color to hover, not none?

Comment thread docs/workflows.md

@janhohenheim janhohenheim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch! Not sure about hover vs none, since I think you can hold press, move the cursor away, and then release? But yeah hover probably makes more sense

Comment thread src/theme/interaction.rs Outdated
return;
};

*bg = palette.none.into();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
*bg = palette.none.into();
*bg = palette.hovered.into();

@olekspickle

olekspickle commented Feb 6, 2026

Copy link
Copy Markdown
Contributor Author

I first made it hover, but then thought hover would take it over afterwards anyway. Glad my first intuition turned out right :D

@janhohenheim
janhohenheim enabled auto-merge (squash) February 6, 2026 11:23
@janhohenheim
janhohenheim merged commit 5add04e into TheBevyFlock:main Feb 6, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants