-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Improved external dialog to no longer extend onto other monitors #3695
Improved external dialog to no longer extend onto other monitors #3695
Conversation
Few things.
|
Changed the check regardless, might be less pretty to look at, but it's not worth creating an offset each time when most of the time it's not used anyways. |
@punker76 or @timunie The For the So the only options im further left with is.
option 2 would not persé fix the issue but would offer more flexibility at the least. Any input would be appreciated |
@AKruimink I have a solution for it and will check in later. |
Use the monitor from where the owner stays and use the working area for the dialog size properties and remove also the left and right border.
… new extension method GetMonitorWorkSize Use this method in both variants SetupExternalDialogWindow and CreateModalExternalWindow which also fixes the main issue without calculating an offset by hand.
@AKruimink your changes have been merged, thanks for your contribution 👍 |
Describe the changes you have made to improve this project
CreateModalExternalWindow
SetupExternalDialogWindow
Use the monitor from where the owner stays and use the working area for the dialog size properties and remove also the left and right border.
GetMonitorWorkSize
Use this method in both variants SetupExternalDialogWindow and CreateModalExternalWindow which also fixes the main issue without calculating an offset by hand.
Closed Issues
Closes #3538