Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-10 Fir de Conversatie Roland Eggner
On 2014-12-08 Monday at 20:07 -0800 mattn wrote: > On Friday, December 5, 2014 10:37:34 PM UTC+9, Roland Eggner wrote: > > Priorities, when “Search”, “CursorLine” and “cterm(fg|bg)” overlap: > > > >│ previously │ with your patch │ IMO even better would be > > ───┼──

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-10 Fir de Conversatie Roland Eggner
On 2014-12-05 Friday at 15:00 -0800 Daniel Hahler wrote: > Roland Eggner wrote: > > On 2014-12-05 Friday at 14:41 +0100 Christian Brabandt wrote: > > > What patch and can you tell us how to reproduce the problem? > > Maybe your webmail user surface can be told to unhide quoted message parts? > > >

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-08 Fir de Conversatie mattn
On Friday, December 5, 2014 10:37:34 PM UTC+9, Roland Eggner wrote: > A: Yes. > Q: Should I reply below the text I am replying to? > > > On 2014-12-04 Thursday at 15:53 -0800 mattn wrote: > > On Friday, December 5, 2014 1:29:21 AM UTC+9, Roland Eggner wrote: > > > On 2014-12-03 Wednesday at 23:

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-05 Fir de Conversatie Bram Moolenaar
Yasuhiro Matsumoto wrote: > Hi list. > > If overlapping ctermbg and CursorLine, background of CursorLine disappears. > > http://go-gyazo.appspot.com/a2f174089dbe322f.png > > This should be below: > > http://go-gyazo.appspot.com/01f192892afb5e9a.png > > Below is a patch. Please check. > > ht

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-05 Fir de Conversatie Daniel Hahler
Roland Eggner wrote: > On 2014-12-05 Friday at 14:41 +0100 Christian Brabandt wrote: > > What patch and can you tell us how to reproduce the problem? > > Maybe your webmail user surface can be told to unhide quoted message parts? > > Patch URL: > 34 lines above … thanks for the relativenumber fe

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-05 Fir de Conversatie Roland Eggner
Hi Christian! On 2014-12-05 Friday at 14:37 +0100 Roland Eggner wrote: > On 2014-12-04 Thursday at 15:53 -0800 mattn wrote: > > On Friday, December 5, 2014 1:29:21 AM UTC+9, Roland Eggner wrote: > > > On 2014-12-03 Wednesday at 23:41 -0800 mattn wrote: > > > > If overlapping ctermbg and CursorLine

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-05 Fir de Conversatie Christian Brabandt
Am 2014-12-05 14:37, schrieb Roland Eggner: I did, but too short: Today I got the first crash without your patch. Thus I re-included it in my patch queue, dropped 2 other recently introduced patches, and so far observed no further crashes. Sorry for suspecting your patch yesterday. Prioriti

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-05 Fir de Conversatie Roland Eggner
A: Yes. Q: Should I reply below the text I am replying to? On 2014-12-04 Thursday at 15:53 -0800 mattn wrote: > On Friday, December 5, 2014 1:29:21 AM UTC+9, Roland Eggner wrote: > > On 2014-12-03 Wednesday at 23:41 -0800 mattn wrote: > > > If overlapping ctermbg and CursorLine, background of C

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-04 Fir de Conversatie mattn
I'm guessing that my patch doesn't depend on the crash. Because I didn't change condition for allocation, memory operation, etc. Did you try without my patch? On Friday, December 5, 2014 1:29:21 AM UTC+9, Roland Eggner wrote: > On 2014-12-03 Wednesday at 23:41 -0800 mattn wrote: > > If overlappi

Re: hl-CursorLine is overwritten by ctermbg/guibg

2014-12-04 Fir de Conversatie Roland Eggner
On 2014-12-03 Wednesday at 23:41 -0800 mattn wrote: > If overlapping ctermbg and CursorLine, background of CursorLine disappears. > > http://go-gyazo.appspot.com/a2f174089dbe322f.png > > This should be below: > > http://go-gyazo.appspot.com/01f192892afb5e9a.png > > Below is a patch. Please chec

hl-CursorLine is overwritten by ctermbg/guibg

2014-12-03 Fir de Conversatie mattn
Hi list. If overlapping ctermbg and CursorLine, background of CursorLine disappears. http://go-gyazo.appspot.com/a2f174089dbe322f.png This should be below: http://go-gyazo.appspot.com/01f192892afb5e9a.png Below is a patch. Please check. https://gist.github.com/mattn/6fb1a3b82f267a8f3633 - Ya