[Bug Fix] Multiple Floating Elements Cannot Use Clay_Hovered()
#461
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
elmfrain
changed the title
Clay_Hovered()Clay_Hovered()
Owner
|
Hello, thanks so much for opening this one! I'll make some small adjustments to your PR then we'll get it merged 🙂 |
Contributor
Author
|
Sounds good! |
Owner
|
Sorry for the super long delay on this! I've just rearranged the struct fields slightly to reduce the padding in |
timwmillard
pushed a commit
to timwmillard/clay
that referenced
this pull request
Oct 13, 2025
…barker#461) Co-authored-by: Nic Barker <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary from issue #459: If a parent element has multiple floating children, then each child cannot use
Clay_Hovered()because Clay will report an error, and hovering won't be handled properly.I've found that
Clay_Hovered()uses an anonymous ID for each button, generated from the amount of sibling elements as the offset. However, since floating children are not logically part of their parents, their IDs will not be unique.My fix makes an element keep track of how many floating children they have to ensure generated anonymous IDs remains unique for each floating child.
clay-bug-fix.mp4