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

List:       kde-panel-devel
Subject:    Re: review of tabbar
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-07-30 21:15:44
Message-ID: 200807301515.44881.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 30 July 2008, Marco Martin wrote:
> on a logistic note: do i move it to kdereview or can ileave it in
> playground/widgets/tabbar ?

moving it to kdereview is that standard mechanism.

> the thing i am most hesitant on it is the way to make the tabbar vertical:
> it has to be done from TabBar::setTabBarShape() otherwise the layout of the
> tabwidget gets broken
> to me if it would be more simple and supporting only horizontal tabbars
> would be more than ok, but the fact that it's possible to set it vertical
> from nativewidget, uhm not good...

this is probably solvable with a bit of creativity. subclass TabBar and track 
the orientation in there; then reimplement tabLayoutChange and check when the 
orientation changes (compare the old to the new) and update the look at that 
point.

however, i'm not exactly sure what you mean by "layout .. get broken". what 
aspect of the layout? do you mean where the widget pages are shown?

> or, even removing the access to nativewidget?

no, that would break the API pattern

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech


["signature.asc" (application/pgp-signature)]

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


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

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