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: "Go to line" does not work sometimes #5328

Closed
mstv opened this issue Aug 19, 2018 · 2 comments
Closed

Commit dialog: "Go to line" does not work sometimes #5328

mstv opened this issue Aug 19, 2018 · 2 comments
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?
bug

What is the current behavior?
Usually, the "Go to line" subdialog can be opened by pressing Ctrl+G.
This does not work if all lines of the diff are visible.
This does not work anymore after Ctrl+G was pressed for an image file or a binary file.

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 changed or new .png file and with a text file with several modifications
  • select the .png file
  • hit Ctrl+G
  • select the text file
  • hit Ctrl+G

What is the expected behavior?
The shortcut key shall be displayed in the context menu.
The context menu text shall be "Go to line... Ctrl+G", i.e. with dots "..." because it will open a subdialog.
The menu item shall be disabled if an image file or a binary diff is displayed.
The menu item and the shortcut shall work if a textual diff is displayed (regardless whether all lines are visible or not).

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)?
didn't check

@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 👓 status: needs review labels Aug 20, 2018
@RussKie
Copy link
Member

RussKie commented Aug 20, 2018

I don't believe it worked even in 2.51 version. At least it doesn't work for me...

IIUC, the code responsible for the dialog is controlled by this method:

public bool IsGotoLineUIApplicable()
{
return _isGotoLineUIApplicable;
}

It gets set here:

_isGotoLineUIApplicable = !isDiff;

RussKie added a commit to RussKie/gitextensions that referenced this issue Aug 20, 2018
Commit dialog: "Go to line" feature did not work for text-based files,
however it was enabled to binary files.

Fixes gitextensions#5328
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
@RussKie
Copy link
Member

RussKie commented Oct 15, 2018

Fixed in #5563

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

No branches or pull requests

2 participants