Abdelrazak Younes wrote:

> Stefan Schimanski wrote:
>> Here is a fix for the grey-bar bug, i.e. randomly only the current
>> paragraph is drawn and everything else is greyed out.
>> 
>> I think it is related to synthetic mouse event. Somehow a redraw is
>> triggered, but the ViewMetric.update_strategy is set to NoScreenUpdate.
>> The two condition, that I change in the patch, evaluate to true and the
>> grey bars are drawn.
>> 
>> Here is a backtrace of the issue:
>> 
>> #0  lyx::paintText ([EMAIL PROTECTED], [EMAIL PROTECTED]) at
>> /Users/sts/Quellen/mac/lyx-devel/src/rowpainter.cpp:1065
>> #1  0x0023570a in lyx::frontend::GuiWorkArea::updateScreen
>> (this=0x261a0e30) at
>> /Users/sts/Quellen/mac/lyx-devel/src/frontends/qt4/GuiWorkArea.cpp:592
>> #2  0x0023574f in lyx::frontend::GuiWorkArea::expose (this=0x261a0e30,
>> x=0, y=31, w=671, h=105) at
>> /Users/sts/Quellen/mac/lyx-devel/src/frontends/qt4/GuiWorkArea.cpp:575
>> #3  0x0022295a in lyx::frontend::WorkArea::redraw (this=0x261a0e44) at
>> /Users/sts/Quellen/mac/lyx-devel/src/frontends/WorkArea.cpp:159
>> #4  0x00223212 in lyx::frontend::WorkArea::dispatch (this=0x261a0e44,
>> [EMAIL PROTECTED], k=none) at
>> /Users/sts/Quellen/mac/lyx-devel/src/frontends/WorkArea.cpp:218
>> #5  0x00237432 in lyx::frontend::GuiWorkArea::mouseMoveEvent
>> (this=0x261a0e30, e=0xbfffefe0) at
>> /Users/sts/Quellen/mac/lyx-devel/src/frontends/qt4/GuiWorkArea.cpp:414
>> 
>> It happened with two footnote in the view, just moving the mouse around.
>> So could it be related to the mouse hover maybe?
> 
> Looks like a good analysis.
> 
> 
>> I think though that the change in the patch is reasonable nevertheless.
>> The comparison to SingleParUpdate is wrong. The drawText function must
>> work with every update_strategy.
> 
> I think you're right Stefan and the problem is most certainly caused by
> a combination of mouse hover and SinglePar when the focus comes back.

SingleParUpdate? I thought it was about NoUpdate. But shouldn't we just
return when vi.update_strategy == NoUpdate? And why are we calling
paintText in the first place?

Sorry if the questions don't make much sense, it's a long time I don't look
at this part of the code... ;-)

A/


Reply via email to