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

TSaveDialog problem #302

Open
aivarsi opened this issue Mar 23, 2023 · 1 comment
Open

TSaveDialog problem #302

aivarsi opened this issue Mar 23, 2023 · 1 comment

Comments

@aivarsi
Copy link

aivarsi commented Mar 23, 2023

When TSaveDialog is used with ofOverwritePromt in Options, the message box that warns about overwriting the existing file is in the wrong size.
I tested it using ThemedSysControls demo application, I just added TSaveDialog in addition to the other dialogs and enabled its ofOverWritePrompt property. Now when you select an existing file in the save dialog, it says "file.ext already exists. Do you want to replace it?". Unfortunately this dialog is not high enough and the Yes/No buttons are cut off.
This happens in all themes except "Windows".

savedialog

@aivarsi
Copy link
Author

aivarsi commented Mar 1, 2024

Additional Information on the Window Sizing Issue:

Upon further investigation, it appears the issue is linked to the handling of the WM_SHOWWINDOW message within Vcl.Styles.Utils.Forms.

Key Observation: The variable DX, used for adjusting the window size, is calculated to an excessively large negative value. This miscalculation significantly diminishes the final window dimensions.

Recommendation:

A review of the DX calculation logic during the WM_SHOWWINDOW message processing may be necessary to correct the window size reduction issue.

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

No branches or pull requests

1 participant