From kde-core-devel  Sun Feb 22 16:48:51 2009
From: Shaun Reich <predator106 () gmail ! com>
Date: Sun, 22 Feb 2009 16:48:51 +0000
To: kde-core-devel
Subject: Re: [PATCH] KTabWidget / KTabBar changes for Qt 4.5
Message-Id: <2165e9910902220848p435a8a4fx41729727a570df96 () mail ! gmail ! com>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=123542321226908

On Sun, Feb 22, 2009 at 10:54 AM, Shaun Reich <predator106@gmail.com> wrote:
> On Sat, Feb 21, 2009 at 8:01 AM, Peter Penz <peter.penz@gmx.at> wrote:
>>
>> Hi Urs,
>>
>> Am Saturday, 21. February 2009 12:58:19 schrieb Urs Wolfer:
>> > Please review the following patch.
>> >
>> > Changes related to Qt 4.5 QTabWidget / QTabBar improvements:
>> > * Deprecate:
>> >   * KTabBar/KTabWidget#isTabReorderingEnabled; use QTabBar#setMovable
>> >   * KTabBar/KTabWidget#setCloseButtonEnabled; use QTabBar#setTabsClosable
>> >   * KTabBar/KTabWidget#setTabCloseActivatePrevious; use
>> > QTabBar#setSelectionBehaviorOnRemove
>> >   * signal KTabBar#moveTab; port to QTabBar#setMovable and connect to
>> > QTabBar#tabMoved
>> >   * signal KTabBar#closeRequest; connect to QTabBar#tabCloseRequested
>> >
>> > Remove all the close icon related code; call instead QTabBar code
>> > internally already.
>>
>> I did not have a detailed look at the deprecated stuff, but I was responsible
>> for adding the close-icon related code for KDE 4.1 -> removing this code is OK
>> :-)
>>
>> (the code was quite hacky and I'm glad that Qt 4.5 supports this natively now)
>>
>> Thanks and bye,
>> Peter
>>
>> > There are some known rendering issues with Qt 4.5 QTabBar which need to be
>> > fixed in Oxygen style. See #184782.
>> >
>> > Bye
>> > urs
>>
>

Okay, I just saw what you have committed this morning, I have just
committed a fix to check to see if the user of a KTabBar has the
QTabBar's->setMovable(true) set, otherwise when they do, we try to use
our own dragging system too, and the tabs that are movable are quite
difficult to move, and very awkward.


--
Riverenter Vestri,
Shaun Reich