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

Commit dialog: retain the view position when updating the diff #5327

Closed
mstv opened this issue Aug 19, 2018 · 2 comments · Fixed by #5563
Closed

Commit dialog: retain the view position when updating the diff #5327

mstv opened this issue Aug 19, 2018 · 2 comments · Fixed by #5563
Assignees
Labels
area: user experience type: regression regression, normally to latest official release
Milestone

Comments

@mstv
Copy link
Member

mstv commented Aug 19, 2018

Do you want to request a feature or report a bug?
incomplete new feature

What is the current behavior?
The commit dialog refreshes automatically when GE gets the focus again since v3-ux.
The cursor/scroll position is reset to the top of the diff.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Open the commit dialog with a diff (regardless staged or unstaged), scroll down the diff, switch to an other application, switch back to GE.

What is the expected behavior?
The view position shall be restored after reloading the diff.
This should be tried, too, when the display options are changed using grafik.

If the cursor position was visible before, it should be restored.
If not, the previous top or the middle line shoud be restored.

Environment you encounter the issue:

  • GitExtensions version: master
  • GIT version: 2.17.1.w.2-x64
  • OS version: Windows 7
  • .NET version: 4.7

Did this work in previous version of GitExtensions (which)?
2.5

@vbjay
Copy link
Contributor

vbjay commented Aug 19, 2018 via email

@mstv
Copy link
Member Author

mstv commented Aug 19, 2018

I appreciate the automatic update in general. But the current behavior annoys me and will do for others which don't know that it can be turned off.

You can consider it a new feature regarding switching on/off the display of the entire file.
As side effect, its implementation will improve the ux with automatic update enabled significantly (at least for me).

@RussKie RussKie added this to the 3.00 milestone Aug 20, 2018
@RussKie RussKie added type: regression regression, normally to latest official release area: user experience labels Aug 20, 2018
mstv added a commit to mstv/gitextensions that referenced this issue Sep 23, 2018
mstv added a commit to mstv/gitextensions that referenced this issue Sep 24, 2018
mstv added a commit to mstv/gitextensions that referenced this issue Sep 25, 2018
@RussKie RussKie modified the milestones: 3.00, 3.00-beta2 Sep 26, 2018
RussKie pushed a commit to RussKie/gitextensions that referenced this issue Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: user experience type: regression regression, normally to latest official release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants