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

Inherit default face for setting background attribute #115

Merged
merged 1 commit into from
Jan 25, 2016

Conversation

syohex
Copy link
Contributor

@syohex syohex commented Jan 25, 2016

This is related to #114.
CC: @xuchunyang

@syohex
Copy link
Contributor Author

syohex commented Jan 25, 2016

Original

before

With this change

after

syohex added a commit that referenced this pull request Jan 25, 2016
Inherit default face for setting background attribute
@syohex syohex merged commit b71d34b into master Jan 25, 2016
@syohex syohex deleted the inherit-default branch January 25, 2016 14:07
seven-beep added a commit to seven-beep/git-gutter that referenced this pull request Sep 15, 2024
     This variable specifies functions called during redisplay when
     window buffers have changed.  The value should be a list of
     functions that take one argument.

     Functions specified buffer-locally are called for any window
     showing the corresponding buffer if that window has been created or
     assigned that buffer since the last time window change functions
     were run.  In this case the window is passed as argument.

I used more naively this hook for +2 years with in mind to contribute back
sometime.  Unfortunately as time passed, I lost the memory of a reproductible
test case.  I think I implemented this after seeing emacsorphanage#115 (Git gutter annotations
disappearing).

As I finally took the time to present you the hook, I realized that it would
mean to obsolete a bunch of functions and variables in the library such as:

 - git-gutter:update-windows-commands
 - git-gutter:pre-command-hook
 - git-gutter:update-other-window-buffers
 - git-gutter:post-command-hook
 - git-gutter:quit-window
 - git-gutter:switch-to-buffer

I also removed `git-gutter-mode` as it is generated by the minor mode and while
testing my changes, its presence caused a small bug of the mode not activating
when it should have.
seven-beep added a commit to seven-beep/git-gutter that referenced this pull request Sep 15, 2024
     This variable specifies functions called during redisplay when
     window buffers have changed.  The value should be a list of
     functions that take one argument.

     Functions specified buffer-locally are called for any window
     showing the corresponding buffer if that window has been created or
     assigned that buffer since the last time window change functions
     were run.  In this case the window is passed as argument.

I used more naively this hook for +2 years with in mind to contribute back
sometime.  Unfortunately as time passed, I lost the memory of a reproductible
test case.  I think I implemented this after seeing emacsorphanage#115 (Git gutter annotations
disappearing).

As I finally took the time to present you the hook, I realized that it would
mean to obsolete a bunch of functions and variables in the library such as:

 - git-gutter:update-windows-commands
 - git-gutter:pre-command-hook
 - git-gutter:update-other-window-buffers
 - git-gutter:post-command-hook
 - git-gutter:quit-window
 - git-gutter:switch-to-buffer

I also removed `git-gutter-mode` as it is generated by the minor mode and while
testing my changes, its presence caused a small bug of the mode not activating
when it should have.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant