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
Fix JumpList being created too soon #8499
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8499 +/- ##
==========================================
+ Coverage 54.75% 54.80% +0.05%
==========================================
Files 898 898
Lines 64251 64255 +4
Branches 11440 11478 +38
==========================================
+ Hits 35179 35218 +39
+ Misses 26390 26333 -57
- Partials 2682 2704 +22
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should work, have not run it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 works for me
Currently, at least on some systems, there is no process-level main window yet when FormBrowse is loaded, so the creation of the JumpList fails ("A valid active Window is needed"). So we wait for the OnActivated(), where this seems to be always OK. We also need to keep track of any call to AddToRecent() it it happened before actual JumpList creation. Fixes gitextensions#8421
81430ee
to
c91a780
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fill in the PR description. We have a template. At least the very helpful "Fixes #8421" and the DCO clause should be there.
Indeed, I created the PR through GitExtensions itself, and it didn't use the template 🙂 Note that the "Fixes" part was already in the commit message, though. |
Thank you |
Fixes #8421
✒️ I contribute this code under The Developer Certificate of Origin.