Skip to content
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: Pull dialog not remember user selection #6615

Conversation

RussKie
Copy link
Member

@RussKie RussKie commented May 20, 2019

If "Open Pull dialog ..." is set as a default pull operation, the pull dialog does not have correct settings preselected upon open.
Also since refactors in v3 pull dialog does not remember user selection (such as Auto stash).

Fixes #6503

(cherry picked from commit 691c45b)

Proposed changes

  • The fix restores FormPullAction setting that remembers user's selection in the dialog.
  • The fix also restores the capture of AutoStash selection.

Screenshots

Before

image

After

image

Test methodology

  • reproduced the issue / compared with v2.51.04
  • manual tests
  • unit tests

Test environment(s)

  • Git Extensions 3.1.1
  • Build 691c45b
  • Git 2.16.1.windows.4
  • Microsoft Windows NT 10.0.17134.0
  • .NET Framework 4.7.3324.0
  • DPI 144dpi (150% scaling)

✒️ I contribute this code under The Developer Certificate of Origin.

If "Open Pull dialog ..." is set as a default pull operation, the pull
dialog does not have correct settings preselected upon open.
Also since refactors in v3 pull dialog does not remember user selection
(such as `Auto stash`).

The fix restores `FormPullAction` setting that remembers user's selection
in the dialog.
The fix also restores the capture of `AutoStash` selection.

Fixes gitextensions#6503

(cherry picked from commit 691c45b)
@codecov
Copy link

codecov bot commented May 20, 2019

Codecov Report

Merging #6615 into release/3.1 will increase coverage by 0.03%.
The diff coverage is 80%.

@@              Coverage Diff               @@
##           release/3.1   #6615      +/-   ##
==============================================
+ Coverage        47.07%   47.1%   +0.03%     
==============================================
  Files              696     696              
  Lines            52401   52448      +47     
  Branches          6917    6923       +6     
==============================================
+ Hits             24667   24707      +40     
- Misses           26393   26395       +2     
- Partials          1341    1346       +5
Flag Coverage Δ
#production 36.48% <33.33%> (ø) ⬆️
#tests 97.57% <100%> (ø) ⬆️

@RussKie RussKie merged commit cc7b3c2 into gitextensions:release/3.1 May 20, 2019
@RussKie RussKie deleted the fix_6503_Pull_dialog_not_remember_user_selection_311 branch October 20, 2020 06:06
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.

None yet

1 participant