- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 46
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
Consider using indicators.el #16
Comments
Do you have a screenshot, to give an idea how that would look? |
I am thinking along the lines of eclipse's "Quick Diff" feature. It dynamically highlights changes relative to the file on disk or a revision in the left fringe and relatively for the whole buffer in the scroll region. It is more intuitive than occur for example. Indicators.el allows the fringe elements to be precisely positioned and sized. You can see some guy's (overly customized) screenshot. Its the best I could find in 5 minutes. Instead of seeing the lines you remove you have only a marker between lines where code used to be. For that reason you only see the code how it is now. Placing the mouse on the fringe where there are highlights (adds, modifies, deletions) shows the diff in a tool tip. However the tool tip diff might be a later step. Its a suggestion as I noticed you were some of the way there already. |
I meant a screenshot of indicators.el in action, but this one also got the idea across. I'll see what it allows to do. |
Sorry, it was closed by a commit in company-mode during a weird broken push. My mistake. |
I've pushed a new minor mode using Note the limitations:
|
Thanks for this chance to give feedback and I'll let you know asap. I'm in Matt On Sun, Jan 19, 2014 at 1:37 PM, Dmitry Gutov [email protected]:
|
Wow, there's something you don't read every day. :) Please take your time. |
I'm looking at the git case on windows. I can see is that you can implement some of the the existing functionality with indicators.el so it does work. Btw I don't know much lisp, but is this a bug?: (dolist (buffer (buffer-list)) Do you think the limitations you mentioned above limit the usefulness of indicators.el for diff-hl at the moment? |
Sorry, I don't understand. Could you rephrase that?
If you mean that this block only executes when
I do. Ideally, Emacs core should implement some sort of layered rendering for fringes, but who knows if we ever get that. |
Don't know if it has been suggested before, but maybe use indicators.el to mark (relative) fringes more precisely with bitmaps.
Also if using indicators.el, don't show deleted lines -- just mark the delete position in fringe too.
The text was updated successfully, but these errors were encountered: