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

🐛 Preview Border has no limit #268

Closed
2 tasks done
Soft-Bred opened this issue Mar 26, 2024 · 2 comments · Fixed by #293
Closed
2 tasks done

🐛 Preview Border has no limit #268

Soft-Bred opened this issue Mar 26, 2024 · 2 comments · Fixed by #293
Assignees
Labels

Comments

@Soft-Bred
Copy link
Collaborator

Bug Description

Found out that the border on the preview does not have a maximum limit, unlike some padding sliders in the app. It's not a huge bug. I think it's pretty funny because nobody in their right mind would ever actually use a border this large, I just think it's funny that it's possible. Do know that anything larger than 200 will actually render quite funnily and you can't even see the preview or the window anymore.

Steps To Reproduce

  1. Set border for Preview to a number larger than 100
  2. Try resize the window with preview.
  3. Laugh.

Expected Behavior

Border has a max limit on its width

Actual Behavior

Border can be set to an infinite amount and actually reflect in the app

Screenshots

006873.mp4

MacOS Version

14.3.1 (23D60)

Loop Version

Version 1.0.0-beta.14 (771)

Additional Context

No response

Final Checks

  • My issue title is descriptive
  • This is a single bug (multiple bugs should be reported individually)
@Soft-Bred Soft-Bred added the Bug label Mar 26, 2024
@MrKai77
Copy link
Owner

MrKai77 commented Mar 26, 2024

This is hilarious, it looks so weird 😂
I will make sure to fix this!

@MrKai77 MrKai77 self-assigned this Mar 26, 2024
@MrKai77
Copy link
Owner

MrKai77 commented Mar 26, 2024

Looks like many other slider options don't have a limit either..
The preview window also looks really funky when its corner radius is set to something really high!

@MrKai77 MrKai77 mentioned this issue Mar 28, 2024
2 tasks
@MrKai77 MrKai77 linked a pull request Mar 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants