> On July 19, 2012, 9:04 a.m., Marco Martin wrote: > > thanks, wouldn;t have remembered ;) > > > > the changes seems good, but i'm not sure about giving kickoff a copy of the > > tabbar. > > > > any reason this is not proposed as a patch for the tabbar component itself? > > Greg T wrote: > Of course I could do that. But I didn't know if such a general change > would be welcome. Maybe people love equally sized blocks? > > Marco Martin wrote: > i really think we shouldn't start to customize the components in a single > applet like that. > > what could be done is: > as implicit width of the tabbar should be a width such that all tabs have > the same width that is the one of the largest tab, so by default it would > attempt to never elide (probably some work in the bindings is needed to > export this as preferred width of the applet). > > if it's forced to be smaller it starts to elide the text. > > Greg T wrote: > I'm not sure if I can follow you, but basically you describe the current > behavior. Every TabButton got a 'implicitHeight: label.paintedHeight' and the > layout calculates a minimum implicitWidth that doesn't elide text. The export > is missing, though.
basically would be kickoff opening by default wide enough to make all text fit (maybe still not going over a maximum width) i'm not sure what's still missing in the bindings to make this feasible tough - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105405/#review16103 ----------------------------------------------------------- On July 1, 2012, 8:42 p.m., Greg T wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105405/ > ----------------------------------------------------------- > > (Updated July 1, 2012, 8:42 p.m.) > > > Review request for Plasma, Marco Martin and Martin Gräßlin. > > > Description > ------- > > Heda, > This patch addresses the layout of the tab bar. The tab buttons are now sized > depending of their text width. > > I just copied the tabbar code from kde-runtime and exchanged taskbarLayout. > My main question is: Can I do this more elegantly without copying TabBar.qml? > > > Diffs > ----- > > plasma/desktop/applets/kickoff/package/contents/ui/KickoffTabBar.qml > PRE-CREATION > plasma/desktop/applets/kickoff/package/contents/ui/Private/TabBarLayout.qml > PRE-CREATION > plasma/desktop/applets/kickoff/package/contents/ui/kickoff.qml 4d53208 > > Diff: http://git.reviewboard.kde.org/r/105405/diff/ > > > Testing > ------- > > > Thanks, > > Greg T > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel