Skip to content
This repository has been archived by the owner on Aug 12, 2018. It is now read-only.

center MessageBox to the window of the program #204

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zufuliu
Copy link
Contributor

@zufuliu zufuliu commented Aug 16, 2017

@XhmikosR center MessageBox to the window of the program.

@XhmikosR
Copy link
Owner

Thanks! Works but the mouse isn't moved to the OK button like in the About dialog.

@zufuliu
Copy link
Contributor Author

zufuliu commented Aug 16, 2017

on win10, I say both OK button is focused / highlighted, mouse isn't moved for both dialog.

@XhmikosR
Copy link
Owner

It is for me. Just change your mouse settings. Then you will see the OK button takes focus in the About dialog while it doesn't in cmd help one.

@zufuliu
Copy link
Contributor Author

zufuliu commented Aug 16, 2017

@XhmikosR keep this open,I will make some changes.

@zufuliu zufuliu force-pushed the master branch 2 times, most recently from d8fd96c to dc972fd Compare August 16, 2017 11:12
@zufuliu
Copy link
Contributor Author

zufuliu commented Aug 16, 2017

@XhmikosR the changes is updated, through the mouse bug not solved. I would identify it as wild behave of MessageBox.
try to find some none existing text in Windows's Notepad, the Not found message box centered on the screen, the OK button does not get mouse.

@zufuliu
Copy link
Contributor Author

zufuliu commented Aug 16, 2017

@XhmikosR snap to default button is done, based on the article at https://blogs.msdn.microsoft.com/oldnewthing/20130826-00/?p=3413/

@zufuliu
Copy link
Contributor Author

zufuliu commented Aug 18, 2017

@XhmikosR can you spend some time to review this PR on weekday?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants