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

feat: Allow 0 height notch #359

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yaxarat
Copy link

@yaxarat yaxarat commented Feb 2, 2025

In this PR

Introduced the ability to set custom notch height that is less than 10.

Closes: #357

Demo

Kapture.2025-02-02.at.14.11.55.mp4

@yaxarat yaxarat changed the title Allow 0 height notch feat: Allow 0 height notch Feb 2, 2025
@yaxarat
Copy link
Author

yaxarat commented Feb 2, 2025

One issue with this change is that when the height is too low (anything below 6), it causes the closing animation to over shoot, leading to a runtime error (does not crash) of Invalid view geometry: height is negative.. I've tried clamping the height, but no luck.

#353 might resolve this though if we force non-wobble animation for small heights.

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.

[FEATURE] enable 0 height option
1 participant