Submodules dialog gets out-of-sync #4365
Comments
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 4, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 6, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 6, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 7, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 10, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 10, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
to gerhardol/gitextensions
that referenced
this issue
Mar 11, 2018
Fixes gitextensions#4564 Fixes gitextensions#4387 Fixes gitextensions#4396 Depends on gitextensions#4562 gitextensions#4663 gitextensions#4365 gitextensions#4366 gitextensions#4367 gitextensions#4368 Submitted to allow review of the outcome to gitextensions#4564 Note: Due to the structure of tests in a separate class, it is very difficult to split this issue in several commits Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (gitextensions#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes A and B revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff
gerhardol
added a commit
that referenced
this issue
Mar 18, 2018
* Browse Diff: Show differences for several selections * Browse Diff tab menu alternatives and presentation Fixes #4564 Fixes #4387 Fixes #4396 Depends on #4562 #4663 #4365 #4366 #4367 #4368 Rewrote RevisionDiffController to make it possible to have some kind of tests. The tests are retrofitted to the current functionality as they are just testing the menu items and not the actions themselves. (The tests adds may more maintenance than they give benefits but the formal test coverage increases.) Browse Diff: Submodules actions available for multi select (#4568) Presents the Reset options for the correct parent in multi select situations No longer differ between parent-child and first-second for Reset Difftool: Describes Selected (B) and First (A) revisions in the menu Difftool arguments depends on parent availability Better detection of parents to A/B for diffs Commit: Limit DiffTool and FileHistory to tracked FormDiff: Limit FileHistory to Tracked Use RevisionDiffController to align to Browse-Diff * Extract common revision diff context menu logic into IRevisionDiffContextMenuController
|
I'm not sure where the problem is, tried to reproduce it, everything seems normal. But apart from that, it is an old issue, no one working on this, we should close it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Do you want to request a feature or report a bug?
-> Bug.
What is the current behavior?
Whenever the list on the left side of the Submodules dialog updates, the last selected item is selected again but the right side of the dialog stays at the first item.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Open "Repository->Submodule..." from the menu. Select a different submodule than the first (you obviously need more than one!). Click the "Synchronize" or "Update" button and wait until it finished.
-> Now the two parts of the dialog do not match anymore.
What is the expected behavior?
The two parts of the dialog shall always match.
Environment you encounter the issue:
Did this work in previous version of GitExtensions (which)?
(unknown).
The text was updated successfully, but these errors were encountered: