Jean-Marc Lasgouttes wrote: > Alfredo> now that drawSelection is separated from draw and called > Alfredo> before it, the background painting of colored insets > Alfredo> overwrites the selection. > > Alfredo> A possible way of solving this is to draw the background in > Alfredo> InsetText::drawSelection instead of InsetText::draw (need to > Alfredo> change LyXText::drawSelection a little bit to work without > Alfredo> knowing the background color) > > Alfredo> Would this be acceptable? > > It seems reasonable. Or you could name that InsetText::drawBackground, > as the selection is in the background.
are you suggesting that we rename the virtual drawSelection to drawBackground in the base inset class? Too many files to change ;-) Alfredo