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 blame control click hold move mouse to the right #6399

Conversation

RussKie
Copy link
Member

@RussKie RussKie commented Mar 24, 2019

Proposed changes

As noted in #6393 (comment) if you "click-hold-move-mouse-to-the-right" to move to the right in
the pane to see the rest of the text, there is no way to ever move back.

The fix handles horizontal scroll for the committer and blocks the hscroll.

Test methodology

  • manual tests to ensure there is no hscroll in the committer control.

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

@ghost ghost assigned RussKie Mar 24, 2019
@ghost ghost added the status: ready label Mar 24, 2019
@RussKie
Copy link
Member Author

RussKie commented Mar 24, 2019

/cc: @pmiossec @jrfnl

If you "click-hold-move-mouse-to-the-right" to move to the right in
the pane to see the rest of the text, there is no way to ever move back.

The fix handles horizontal scroll for the committer and blocks the hscroll.
@RussKie RussKie force-pushed the fix_blame_control_click-hold-move-mouse-to-the-right branch from 60a6fd9 to 0a2869f Compare March 24, 2019 21:21
@pmiossec
Copy link
Member

That fixes the bad behavior. Well done!

@jrfnl
Copy link

jrfnl commented Mar 25, 2019

Tested, not sure if this was the intended behaviour, but basically "click-hold-mouse button-move right" now has no effect at all any more.

This, quite effectively, solves the problem of not being able to move back.

As for not being able to see what's "hidden" from view in the left pane: as the screen divider can be moved, that information can be accessed in that manner.

So, I'm OK with this change.

@RussKie
Copy link
Member Author

RussKie commented Mar 25, 2019

As for not being able to see what's "hidden" from view in the left pane: as the screen divider can be moved, that information can be accessed in that manner.

That is the intended behaviour, if a user requires to see more or less information for the committer - they can use the divider.

@RussKie RussKie merged commit 9efb259 into gitextensions:master Mar 25, 2019
@RussKie RussKie deleted the fix_blame_control_click-hold-move-mouse-to-the-right branch March 25, 2019 07:05
@ghost ghost removed the status: ready label Mar 25, 2019
@RussKie
Copy link
Member Author

RussKie commented Mar 25, 2019

Thank you for testing.

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

Successfully merging this pull request may close these issues.

None yet

3 participants