Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix quote repost hitbox #538

Merged
merged 3 commits into from
Dec 4, 2024
Merged

Fix quote repost hitbox #538

merged 3 commits into from
Dec 4, 2024

Conversation

jb55
Copy link
Contributor

@jb55 jb55 commented Dec 4, 2024

The response from the wide rendered note was incorrect, leading to and
incorrectly sized hitbox. This fixes that.

Additionally, we include note options and note parent into the hitbox
key, as this may influence the size of the note.

Before After

Fixes: #519
Closes: #537
Changelog-Fixed: Fix broken quote repost hitbox
Signed-off-by: William Casarin [email protected]

ksedgwic and others added 3 commits December 3, 2024 09:09
--features debug-widget-callstack
  Show callstack for the current widget on hover if all modifier keys
  are pressed down

--features debug-interactive-widgets
  Show an overlay on all interactive widgets

Notes:
- debug-widget-callstack asserts `egui:callstack` feature when enabled
- Only works in debug builds, compile error w/ release builds
This was causing hitbox issues

Signed-off-by: William Casarin <[email protected]>
The response from the wide rendered note was incorrect, leading to and
incorrectly sized hitbox. This fixes that.

Additionally, we include note options and note parent into the hitbox
key, as this may influence the size of the note.

Before: https://cdn.jb55.com/s/b2464c22a65adb12.png
After: https://cdn.jb55.com/s/52545564d98d278e.png

Fixes: #519
Closes: #537
Changelog-Fixed: Fix broken quote repost hitbox
Signed-off-by: William Casarin <[email protected]>
@jb55 jb55 merged commit 78210e8 into master Dec 4, 2024
8 checks passed
@jb55 jb55 deleted the hitbox-fix branch December 4, 2024 02:00
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.

hitbox does not work consistently on quote re-post
2 participants