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

Scrolling over diff/log jumps around #105

Open
remram44 opened this issue Oct 23, 2020 · 4 comments
Open

Scrolling over diff/log jumps around #105

remram44 opened this issue Oct 23, 2020 · 4 comments
Labels
bug entry level Marks issues that can be easily addressed by new developers just starting with this project. help wanted

Comments

@remram44
Copy link

remram44 commented Oct 23, 2020

I think some of this is intended, but it's really annoying.

When scrolling down past the end of the log, the view changes to diff. When scrolling down past the end of the diff, the view jumps UP to the NEXT commit. Even worse, scrolling back up goes again to the NEXT commit.

It's a bit annoying to be scrolling down a diff and to suddenly have the view jump to a different commit.

There are additional glitches when switching from log view to diff view that way (scrolling down), it won't be possible to scroll through the diff (have to stop scrolling my mousewheel for a few seconds before scrolling works again).

I would appreciate an option to disable this whole thing, though I suspect it could be removed altogether.

@tibirna tibirna added bug entry level Marks issues that can be easily addressed by new developers just starting with this project. help wanted labels Oct 24, 2020
@martenjj
Copy link
Contributor

Don't know whether this is intended behaviour, but it seems that you can stop scrolling switching from log to diff, and scrolling down from diff going to a different commit, by turning off "Show smart labels" in Settings - Browse.

@remram44
Copy link
Author

Well that's embarrassing... 😅

I had noticed the option for tabbed view, but not that particular checkbox. Though it could probably be made more explicit about what it does (it's not clear what "smart labels" are, and the scrolling behavior is not mentioned), unchecking it has fixed the problem for me.

Thank you!

@WickedSmoke
Copy link
Contributor

I agree that this behavior is bothersome and also had no idea that it could be disabled. Yes, at a minimum the name and/or tool tip should mention scrolling.

This seems like a deeper design problem. I'd really like the Log to be visible along side the Diff and not hide it.

@WickedSmoke
Copy link
Contributor

After thinking on it a bit, simply unifying the Log & Diff tabs would be my preferred layout. Similar to how "git show" or gitweb display commits. This would streamline the interface (no tabs or smart labels needed) and the scrolling option could then be called something like "Allow mouse wheel to change commits".

Some option combinations seem useless. If both "Show tabbed revision msg / diff pane" & "Show smart labels" are disabled then the current view (with a now hidden tab) is locked and there is no way to see the other Log/Diff content.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug entry level Marks issues that can be easily addressed by new developers just starting with this project. help wanted
Projects
None yet
Development

No branches or pull requests

4 participants