Size the export dialog to its options and to the monitor it opens on - #100
Merged
Conversation
The dialog opened at a fixed height, which left the bottom empty with a short option list and, on a monitor shorter than that height, put the buttons off the screen: the cap it used was the primary monitor's work area, the wrong screen whenever the board is on another. The height now follows the options column, the preview takes the same height rather than driving it, and the cap is the work area of the monitor the board is on, past which the options scroll. When a change of format regrows the window in place, it is moved back inside the screen. Co-Authored-By: Claude Fable 5.1 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The Export dialog opened at a fixed height. With the PowerPoint options that left a band of nothing under the switches, and on a monitor shorter than the dialog the buttons were off the screen, because the cap was taken from the primary monitor's work area, which is the wrong screen whenever the board is on another.
What changes
MonitorStartupPlacement.WorkAreaHeightreads the work area of the monitor under the owner window, in the window's own units, and the dialog never exceeds it; past that the options scroll.KeepWithinWorkAreamoves it back inside the work area.Verified by opening the built application, pasting a board, and opening the dialog: it ends where the options end, with no scrollbar and no empty band.
🤖 Generated with Claude Code