> On March 21, 2013, 6:54 p.m., David Edmundson wrote:
> > I'm very much against shipping any patches if we don't know why it fixes 
> > it. From what I can see from declarative/declarativeitemcontainer.cpp it 
> > just sets the minimumWidth to it... which I would have thought it would be 
> > doing anyway when we set the minimumWidth.
> > 
> > Any documentation on implicitWidth, preferredWidth, minimumWidth and width 
> > would be incredibly useful. 
> > 
> > Given the proximity to the deadline and your testing I can't see this 
> > breaking anything, so ship it if you're confident with it.
> 
> Marco Martin wrote:
>     meh, browser ate my previous comment :/
>     i was saying: declarativeitemcontainer.cpp is used only when the item is 
> in a Dialog (in the popup)
>     the root qml item is contained in a Plasma::DeclarativeWidget from 
> libplasma
>     
>     the minimum/implicit/maximum sizes of the root qml item gets exported as 
> minimumSize,preferredSize and maximumSize respectively of the 
> qgraphicswidget, therefore minimum,preferred and maximum of the Applet that 
> is in the linear layout of the panel.
>     
>     so, the documentation that applies is
>     http://qt-project.org/doc/qt-4.8/qgraphicslayoutitem.html
>     http://qt-project.org/doc/qt-4.8/qgraphicslinearlayout.html
>     
>     
>     basically minimum width: the layout will never resize the applet less 
> than that, maximum width: the layout will never resize the applet more than 
> that
>     
>     preferred (in qml they decided to rename it to implicit but same thing) 
> the layout will try to give it that size, but will still give it a different 
> size when needed
>     
>     so if as minimum, implicit and maximum you have 12, 56, 100 respectively, 
> the layout will try to make it 56 pixels wide, but can decide to give it a 
> different width, but never less than 12 and never more than 100
> 
> Aleix Pol Gonzalez wrote:
>     I'm still not convinced though about why does this solve the problem, 
> since it's only aliasing the preferred sizes...
>     
>     Also I still have the problem that sometimes the Icon Tasks collapses my 
> plasmoid into the minimum.
> 
> Marco Martin wrote:
>     it's not a problem, it's behaving exactly as you are telling it to...
>     
>     again, because it's the preferred size, you're asking to the panel 
> qgraphicslinearlayout "please make me this width if you can" so usually 
> solves the problem, becausethe panel can comply.
>     
>     in case the icon tasks grows to much, being that just a "preferred" the 
> layout makes more space for the taskbar, shriking this applet as much it can, 
> and as much it can is its "minimum size".
>     now, if you want your plasmoid to be a given size, never less never more 
> without having the layout freedom to screw with it, you have to set a minimum 
> width *and* a macimum width that are exactly the same
> 
> David Edmundson wrote:
>     based on these comments it sounds like our preferredWidth property isn't 
> doing anything.
>     
>     So rather than aliasing like this patch does we should be just renaming 
> the preferredWidth to implicitWidth.
>     
>     Right?

yes, implicit is to be used in place of preferred in the qml part


- Marco


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109648/#review29651
-----------------------------------------------------------


On March 21, 2013, 5:37 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109648/
> -----------------------------------------------------------
> 
> (Updated March 21, 2013, 5:37 p.m.)
> 
> 
> Review request for Plasma, Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> Implements those 2 properties, aliasing to the similar preferredWidth/Height.
> 
> When I have Icon-Only tasks + Chat plasmoid in a panel, their growth rendered 
> unpredictable. Implementing the implicit seems to solve this problem.
> 
> 
> Diffs
> -----
> 
>   chat/org.kde.ktp-chat/contents/ui/main.qml e11269e 
> 
> Diff: http://git.reviewboard.kde.org/r/109648/diff/
> 
> 
> Testing
> -------
> 
> I was reproducing the problem regularly since a couple of days, then now I 
> cannot reproduce anymore.
> 
> There's no testing per se, though...
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to