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

Vertical ruler position not correctly read from settings #6391

Closed
crypto-rsa opened this issue Mar 21, 2019 · 5 comments
Closed

Vertical ruler position not correctly read from settings #6391

crypto-rsa opened this issue Mar 21, 2019 · 5 comments
Milestone

Comments

@crypto-rsa
Copy link
Contributor

Current behaviour

When opening a new diff window (like a file history), the vertical ruler position is not updated according to the settings - it has the default value.

Expected behaviour

The ruler should have correct position right from the start.

Steps to reproduce

  1. Set a non-default value for the vertical ruler position (in Settings - Detailed - Diff Viewer).
  2. Open the history of any file.
  3. The ruler is at the default position.

The ruler only moves to the correct position after clicking the Settings icon in the diff view and clicking OK in the settings window (no editing needed), which suggests the settings are not loaded correctly the first time the window opens.

Did this work in previous version of GitExtensions

Did not check.

Environment

  • Git Extensions 3.1.0
  • Build 7a5680d
  • Git 2.21.0.windows.1
  • Microsoft Windows NT 10.0.17134.0
  • .NET Framework 4.7.3260.0
  • DPI 96dpi (no scaling)
@RussKie
Copy link
Member

RussKie commented Mar 21, 2019

Please test #6392 and confirm the fix

@RussKie RussKie added this to the 3.1.0 milestone Mar 21, 2019
@thimmy687
Copy link
Contributor

thimmy687 commented May 8, 2019

Hi the same issue is in the commit window

my setting is on 120 but on opening the commit window the line is displayed at the default position.

If I open and close the settings without changing something the line moves to the correct position.

before:
image
and after:
image

Here is the version info:

  • Git Extensions 3.0.2.5232
  • Build 23b6f51
  • Git 2.20.0.windows.1
  • Microsoft Windows NT 10.0.16299.0
  • .NET Framework 4.7.3394.0
  • DPI 96dpi (no scaling)

@RussKie
Copy link
Member

RussKie commented May 8, 2019

@thimmy687 please try the 3.1-RC1 and see if it was fixed

@thimmy687
Copy link
Contributor

I will check this, thx

@thimmy687
Copy link
Contributor

Hi it's working well now :)

but it leads me to another issue 😅 which I will report separatly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants