dialog.showMessageBoxSync defaultId not work under windows OS.

[ForJason]

Preflight Checklist

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for an issue that matches the one I want to file, without success.

Issue Details

• Electron Version:
  • 8.0.0

• Operating System:
  • Windows 10 (18362)
  • Ubuntu 14.04 x64

• Last Known Working Electron version:

Expected Behavior

dialog.showMessageBoxSync(null, { type:'warning', buttons: ['Yes', 'No'], defaultId: 1, cancelId: 1, title: 'Power Control', message: "Warning!", });
I set "defaultId:1", after executing the function,it should focus on the "No" button.

Actual Behavior

Execute the function, in windows 10, it focuses on the "Yes" button. And it focuses on the "No" button in Ubuntu 14.04.

To Reproduce

$ git clone git@github.com:ForJason/electron-smt.git
$ cd electron-smt
$ npm start

Screenshots

Additional Information

[codebytere]

@ForJason can you confirm the last working version of this? Did it work in v7? v6, or v5?

[ForJason]

@ForJason can you confirm the last working version of this? Did it work in v7? v6, or v5?
I tested it based on v7.1, v6.1 and v5.0, it did't work. In addition, the v5.0 not support "showMessageBoxSync", replace it with "showMessageBox".
Hope it helps you.

[jado66]

I think this is still an issue.

[electron-triage]

The Electron version reported on this issue is no longer supported. See our supported versions documentation.

If you're still experiencing this issue on a supported version, please open a new issue with an updated repro - a Fiddle is very appreciated.

Electron has a large issues backlog. To help our team prioritize, we're closing older issues and asking for new issues with updated repro steps if it affects a supported version. This helps sort what issues are still relevant and helps us fix them more quickly.

Thanks for your patience and understanding!

[pushkin-]

This repros with Electron 12.0.1. If you explicitly set noLink to false, the issues goes away. Yes and No buttons seems to be defaulting to the no link display. If you change the labels, it will also work, but will look like this (which is where the noLink property comes in):

[pushkin-]

This might affect other common buttons

[MRayermannMSFT]

I see defaultId working even with noLink: true.

[ArnieGA]

Confirming that it still doesn't work as expected

Property	Value
Electron	19.0.4 and 19.0.5
OS Edition	Windows 10 Pro
OS Version	21H2
OS build	19044.1766

The defaultId is simply ignored.

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[ArnieGA]

bump

…

On Fri, Oct 7, 2022, 10:20 PM github-actions[bot] ***@***.***> wrote: This issue has been automatically marked as stale. *If this issue is still affecting you, please leave any comment* (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron <https://www.electronjs.org/releases/stable> or in the beta <https://www.electronjs.org/releases/beta>—please include it with your comment! — Reply to this email directly, view it on GitHub <#22600 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADEPLHIZHN2GBUZTJCAV7XTWCDK77ANCNFSM4LEAHNBA> . You are receiving this because you commented.Message ID: ***@***.***>

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[pushkin-]

bump

[rykermcintyre]

This is still an issue. I have noLink: false, but I also have a non-link button (since it says "OK"). My defaultId is 1, so I would expect "OK" to be selected, but the link button is, instead.

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[pushkin-]

repros in v24

[Andrew-web-coder]

defaultId option is not working at all

[electron-issue-triage]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[MRayermannMSFT]

bump

[electron-issue-triage]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[pushkin-]

bump

[electron-issue-triage]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[pushkin-]

bump

[electron-issue-triage]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[pushkin-]

bump