> On Sept. 17, 2014, 6:35 p.m., Marco Martin wrote: > > hmm, not sure is the correct way, setting implicitsizes should be expected > > to work correctly and not be rewrittten by the component. > > > > maybe it should check at componentcomplete if at that point impolicitWidth > > has been set by the user > > David Edmundson wrote: > I think your patch was right. Consider a Text Item. The implicitSize > changes whenever the text changes.
perhaps.. so it should be as Text, implicitWidth/height become read only, to give an error if the user tries to write the proeprties - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120248/#review66766 ----------------------------------------------------------- On Sept. 17, 2014, 6:22 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120248/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2014, 6:22 p.m.) > > > Review request for Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > Override the implicitSizes in the ruler completley > > Needed to fix: https://gerrit.vesnicky.cesnet.cz/r/#/c/41/2 > Though I think that change was correct (and so is this one) > > > Diffs > ----- > > desktoppackage/contents/configuration/panelconfiguration/Ruler.qml 6cdccc9 > > Diff: https://git.reviewboard.kde.org/r/120248/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
