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

Magit Ediff integration problems #168

Open
kjmph opened this issue Jan 30, 2020 · 6 comments
Open

Magit Ediff integration problems #168

kjmph opened this issue Jan 30, 2020 · 6 comments
Labels
bug a feature isn't working

Comments

@kjmph
Copy link

kjmph commented Jan 30, 2020

Hello, thanks for the great project! However, there seems to be a bug when interacting with Magit and Ediff. If directly calling magit-ediff-show-working-tree, then git-gutter shows the gutter in ediff-window-A. Like so:

Screen Shot 2020-01-30 at 11 54 22 AM

I worked around this by binding a key to this function:

(defun magit-ediff-git-gutter-fix ()
  (interactive)
  (global-git-gutter-mode 0)
  (magit-ediff-show-working-tree (buffer-file-name))
  (global-git-gutter-mode 1))

This works for me, since 90% of my use cases are looking at the current buffer. This results in this output in my example:

Screen Shot 2020-01-30 at 11 54 38 AM

I don't have any pointers on why this is so. Is there anything else I can provide?

Thanks again!

@gonewest818 gonewest818 added the bug a feature isn't working label Mar 24, 2020
@gonewest818
Copy link
Collaborator

I'm not seeing the same behavior as you are:

  • in Emacs 25.3, the "B" buffer is annotated with git-gutter signs.
  • in Emacs 26.3 and 27.0.90, neither buffer gets git-gutter signs.

Can you confirm which version of Emacs you're using and if this is still an issue?

@gonewest818 gonewest818 added need repro we can't reproduce this and removed bug a feature isn't working labels Mar 24, 2020
@kjmph
Copy link
Author

kjmph commented Sep 17, 2020

Hello, sorry for my delay this summer. I'm using GNU Emacs 27.1. I checked, and this still happens. I'm happy to help you reproduce this, what could I provide for you?

@gonewest818
Copy link
Collaborator

Huh, well perhaps it's due to some other difference in the packages you and I are using (besides this one). It can be frustrating and a bit of a grind narrowing that down. But first let me try reproducing again with 27.1 and then see what we learn?

@gonewest818
Copy link
Collaborator

Ok, well the good news is I can now reproduce this (in 27.1) so I don't think I need more from you at this point... I just need time to dig into the reasons why.

@gonewest818 gonewest818 added bug a feature isn't working and removed need repro we can't reproduce this labels Sep 18, 2020
@gonewest818
Copy link
Collaborator

gonewest818 commented Sep 18, 2020

Playing around a little more, I find I can get the gutter on the "B" side of the ediff by hitting a hotkey that causes the layout to rearrange ("?" for help, "|" to toggle vertical or horizontal layout of the two views). Or if you "~" to swap the contents of the views that also works, the gutter draws on the correct side and keeps drawing on the correct side even if you swap the views some more. This isn't a fix, but it's a place for me to look.

@kjmph
Copy link
Author

kjmph commented Sep 18, 2020

Thanks! I can confirm that also fixes the views by changing the layout. I'm glad you are on the case. Let me know if I can help at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug a feature isn't working
Development

No branches or pull requests

2 participants