[BUG] On shadow textbox frame

Gay, Matthieu matthieu.gay at capgemini.com
Mon Sep 16 02:20:40 PDT 2013


The patch following is not good, you can see the bug and the good patch here:


-----Message d'origine-----
De : libreoffice-bounces+matthieu.gay=capgemini.com at lists.freedesktop.org [mailto:libreoffice-bounces+matthieu.gay=capgemini.com at lists.freedesktop.org] De la part de Gay, Matthieu
Envoyé : jeudi 12 septembre 2013 10:53
À : Cedric Bosdonnat
Cc : LibreOffice dev
Objet : RE:[BUG] On shadow textbox frame

Sorry, I've just saw your answer.
In fact I have two bugs:

- The first one is about reading a "*.doc" (see the attached file SIMPLE_TEST.doc), which has the shadow in the foreground.

I resolve this by applying the following patch (Version 3.5.7):

 diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx
index 1677207..796497a 100644
--- a/sw/source/filter/ww8/ww8par6.cxx
+++ b/sw/source/filter/ww8/ww8par6.cxx
@@ -2360,7 +2360,7 @@ bool SwWW8ImplReader::IsDropCap()  bool SwWW8ImplReader::StartApo(const ApoTestResults &rApo,
     const WW8_TablePos *pTabPos)
-    if (0 == (pWFlyPara = ConstructApo(rApo, pTabPos)))
+    if (0 != (pWFlyPara = ConstructApo(rApo, pTabPos)))
         return false;
     // <WW8SwFlyPara> constructor has changed - new 4th parameter
I will put a new bug and the commit, but can you tell me if my patch will provoke any regressions?

- The second one is the bug you answered, I will look in paintfrm.cxx.

Thanks very much for your answers.

Matthieu GAY

De : Cedric Bosdonnat [cbosdonnat at suse.com] Envoyé : lundi 15 juillet 2013 09:56 À : Gay, Matthieu Cc : LibreOffice dev Objet : Re: [BUG] On shadow textbox frame

Hello Matthieu,

On Fri, 2013-07-12 at 13:03 +0000, Gay, Matthieu wrote:
> When I have a text box with shadow on the side, if the background of 
> the frame is transparent (more than 0%) the shadow come to the 
> foreground.
> Con you tell me if this a bug ? If so where can I resolve it? If this 
> a UI bug?

That really sounds like a bug in the Frame painting. I would start looking at SwFlyFrm::Paint:


I hope this helps,
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

More information about the LibreOffice mailing list