[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    [Panel-devel] Svg::elementRect and theme configs
From:       "Leo Franchi" <lfranchi () gmail ! com>
Date:       2007-07-31 22:46:11
Message-ID: c5793d500707311546o4e6a67buef34e82dccc509b3 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hey plasma guys,

Two things. First of all, I've been working with Plasma::Svg, and especially
Svg::elementRect, and after some investigation I figured out why the element
rect returned by the function was actually *outside* the applet itself. It
turns out boundsForElement( QString ) returns the element rect, but "The
transformation matrix of parent elements is not affecting the bounds of the
element." (from
API). So basically in order to get the correct element rect, i had to
add the two lines included in the patch.

Also, regarding the current Theme->setApplicationName(), currently if you
set the application to, say, "amarok", plasma will look for the
[Theme-amarok] group in plasmarc. Now, as Amarok doesn't depend on kdebase,
there might not be a plasmarc at all. So it makes little sense for us to
edit/create plasmarc if none exists. I think it would be a better idea to
look into the application-default config, so amarokrc would hold the
relevant theme data. What do you guys think?

leo

-- 
______________________________________________________
Leo Franchi                    angel666@myrealbox.com
4305 Charlemagne Ct         lfranchi@gmail.com
Austin                                 cell: (650) 704 3680
TX, USA                              home: (650) 329 0125

[Attachment #5 (text/html)]

Hey plasma guys,<br><br>Two things. First of all, I&#39;ve been working with \
Plasma::Svg, and especially Svg::elementRect, and after some investigation I figured \
out why the element rect returned by the function was actually *outside* the applet \
itself. It turns out boundsForElement( QString ) returns the \
element&nbsp;rect,&nbsp;but&nbsp;&quot;The transformation matrix of parent elements \
is not affecting the bounds of the element.&quot; (from \
API).&nbsp;So&nbsp;basically&nbsp;in&nbsp;order&nbsp;to&nbsp;get&nbsp;the&nbsp;correct \
&nbsp;element&nbsp;rect,&nbsp;i&nbsp;had&nbsp;to&nbsp;add&nbsp;the&nbsp;two&nbsp;lines&nbsp;included&nbsp;in&nbsp;the&nbsp;patch.
 <br> <br>Also, regarding the current Theme-&gt;setApplicationName(), currently if \
you set the application to, say, &quot;amarok&quot;, plasma will look for the \
[Theme-amarok] group in plasmarc. Now, as Amarok doesn&#39;t depend on kdebase, there \
might not be a plasmarc at all. So it makes little sense for us to edit/create \
plasmarc if none exists. I think it would be a better idea to look into the \
application-default config, so amarokrc would hold the relevant theme data. What do \
you guys think?  <br><br>leo<br clear="all"><br>-- \
<br>______________________________________________________<br>Leo \
Franchi&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a \
href="mailto:angel666@myrealbox.com" target="_blank" onclick="return \
top.js.OpenExtLink(window,event,this)"> angel666@myrealbox.com</a><br>4305 \
Charlemagne Ct&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <a \
href="mailto:lfranchi@gmail.com" target="_blank" onclick="return \
top.js.OpenExtLink(window,event,this)"> lfranchi@gmail.com</a> \
<br>Austin&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbs \
p;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
cell: (650) 704 3680 <br>TX, \
USA&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp \
;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;home: \
(650) 329 0125  


["plasma.patch" (application/octet-stream)]

_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic