help request on proposed VCL code patch
Lionel Elie Mamane
lionel at mamane.lu
Mon Dec 14 06:13:57 PST 2015
On Wed, Nov 11, 2015 at 10:59:46AM +0000, Caolán McNamara wrote:
> On Fri, 2015-10-30 at 11:31 +0100, Lionel Elie Mamane wrote:
>> There is patch attached to tdf#94069, nominally a rather annoying
>> Base UI bug (crash!), but which touches the VCL event loop (not
>> specific to Base) in a way that I don't understand the consequences
> If I look at the behaviour in an older version of LibreOffice pre idle
> tweaking then on moving that widget it "jumps" around a lot and is
> shown rendering in the original location and then the new one and so
> on. So it didn't really work right in the first place as far as I can
> Looking at the UnoControl PushButton etc which don't suffer from this
> problem I see that when setDesignMode is called for everything else
> they set the "real" widget to hidden (it gets drawn via a different
> mechanism than the real Paint in this case) so when its moved around it
> doesn't generate Invalidate events.
> So, I think the right solution may be to make the GridControl behave
> like the other UnoControl widgets in that design mode turns off
> visibility. So I've done that as
> a7816853bad55ada597092c16ba9a0a761e067d0 and it seems to work well.
> I don't really "use" these things day to day, so testing appreciated
> for this and follow up 44daaebf835bb60fb7e442e928cd30191f15af52 to
> tweak how the grid control is laid out, to ensure I haven't busted
> anything horribly.
Well, you did, but apparently we didn't catch up on it and now we are
about to ship 5.0.4 with a significant regression compared to 5.0.3
Do you think you can take a look at it "fast" or should we revert?
More information about the LibreOffice