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 window position / size does not fit in primary display #850

Merged
merged 7 commits into from
Feb 7, 2025

Conversation

webfiltered
Copy link
Contributor

@webfiltered webfiltered commented Feb 7, 2025

┆Issue is synchronized with this Notion page by Unito

@webfiltered webfiltered requested a review from a team as a code owner February 7, 2025 03:53
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 7, 2025
@webfiltered webfiltered changed the title Window restore state Fix window position / size does not fit in primary display Feb 7, 2025
@webfiltered webfiltered enabled auto-merge (squash) February 7, 2025 03:54
@webfiltered webfiltered disabled auto-merge February 7, 2025 22:26
@webfiltered webfiltered merged commit a9c48ca into main Feb 7, 2025
7 checks passed
@webfiltered webfiltered deleted the window-restore-state branch February 7, 2025 22:26
@christian-byrne
Copy link
Contributor

Do you want to switch to using lodash's clamp? It has handling for NaN and undefined and some perfomance optimizations.

@webfiltered
Copy link
Contributor Author

Current function was just to make it more legible; happy to throw it away.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Windows app goes off screen sometimes
2 participants