Skip to content

Conversation

@mgrojo
Copy link
Member

@mgrojo mgrojo commented Aug 29, 2020

Remove the default and auto-default properties of the cancel button in the
"Fetching data..." dialog.

See issue #2383

Remove the default and auto-default properties of the cancel button in the
"Fetching data..." dialog.

See issue #2383
@mgrojo mgrojo requested a review from justinclift August 29, 2020 15:00
@mgrojo
Copy link
Member Author

mgrojo commented Aug 29, 2020

@justinclift Do you mind launching a Windows build so @datvm can test whether #2383 is solved before merging this?

@mgrojo mgrojo marked this pull request as draft August 29, 2020 15:02
@justinclift
Copy link
Member

justinclift commented Aug 29, 2020

@datvm
Copy link

datvm commented Aug 30, 2020

Hi, just tested @justinclift 's zip file, can confirm the mouse cursor no longer moves to the dialog.

P.s: probably unrelated but an old message box reappears in this build. I think just because you haven't merged the fix yet but still notify you just in case:

image

@mgrojo mgrojo marked this pull request as ready for review August 30, 2020 22:06
@mgrojo mgrojo merged commit c791a9f into master Aug 30, 2020
@mgrojo
Copy link
Member Author

mgrojo commented Aug 30, 2020

Thanks @datvm. It's already merged. The message is already fixed in the master branch.

@mgrojo mgrojo deleted the issue_2383 branch August 30, 2020 22:10
@mgrojo mgrojo restored the issue_2383 branch October 14, 2023 21:27
@mgrojo mgrojo deleted the issue_2383 branch October 14, 2023 21:45
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

Successfully merging this pull request may close these issues.

3 participants