This repository has been archived by the owner. It is now read-only.

center MessageBox to the window of the program #204

Open
wants to merge 2 commits into
base: master
from

Conversation

2 participants
@zufuliu
Contributor

zufuliu commented Aug 16, 2017

@XhmikosR center MessageBox to the window of the program.

@XhmikosR

This comment has been minimized.

Show comment
Hide comment
@XhmikosR

XhmikosR Aug 16, 2017

Owner

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

Owner

XhmikosR commented Aug 16, 2017

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

@zufuliu

This comment has been minimized.

Show comment
Hide comment
@zufuliu

zufuliu Aug 16, 2017

Contributor

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

Contributor

zufuliu commented Aug 16, 2017

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

@XhmikosR

This comment has been minimized.

Show comment
Hide comment
@XhmikosR

XhmikosR Aug 16, 2017

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.

Owner

XhmikosR commented Aug 16, 2017

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

This comment has been minimized.

Show comment
Hide comment
@zufuliu

zufuliu Aug 16, 2017

Contributor

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

Contributor

zufuliu commented Aug 16, 2017

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

@zufuliu

This comment has been minimized.

Show comment
Hide comment
@zufuliu

zufuliu Aug 16, 2017

Contributor

@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.

Contributor

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

This comment has been minimized.

Show comment
Hide comment
@zufuliu

zufuliu Aug 16, 2017

Contributor

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

Contributor

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

This comment has been minimized.

Show comment
Hide comment
@zufuliu

zufuliu Aug 18, 2017

Contributor

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

Contributor

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.