Hi Bram!

On So, 05 Jun 2011, Bram Moolenaar wrote:

> 
> ZyX wrote:
> 
> > > The diff filler means that this line doesn't exist in that buffer.
> > > There can't possibly be a sign there, so why do you expect the sign
> > > column to show up there?
> > Just for the same reason why I see number column there (though with no 
> > numbers). 
> > I think my screen shot is clear enough: here is
> >     FFSStext
> >     FF------
> >     FFSStext
> > (F is fold column, S is sign column)
> > If I enabled numbers it would be more clear:
> >     FFSSNNN text
> >     FF    ------
> >     FFSSNNN text
> > which I guess is really
> >     FFSSNNN text
> >     FFnnn ------
> >     FFSSNNN text
> > (where N is a digit and `n' is a space in place of absent digit).
> > 
> > Here is screenshot with numbers enabled:
> > http://img-fotki.yandex.ru/get/4705/kp-pav.1/0_695a2_66d73e36_orig.png
> 
> Yeah, that doesn't look right.

I think, this patch fixes it:

diff --git a/src/screen.c b/src/screen.c
--- a/src/screen.c
+++ b/src/screen.c
@@ -3399,11 +3399,7 @@
                draw_state = WL_SIGN;
                /* Show the sign column when there are any signs in this
                 * buffer or when using Netbeans. */
-               if (draw_signcolumn(wp)
-# ifdef FEAT_DIFF
-                       && filler_todo <= 0
-# endif
-                  )
+               if (draw_signcolumn(wp))
                {
                    int text_sign;
 # ifdef FEAT_SIGN_ICONS



regards,
Christian

-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

Raspunde prin e-mail lui