Bug 193893 - Tabs in Koqueror are displayed improperly (regression)
Summary: Tabs in Koqueror are displayed improperly (regression)
Status: RESOLVED FIXED
Alias: None
Product: Oxygen
Classification: Plasma
Component: style (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Camilla Boemann
URL:
Keywords:
: 194377 201903 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-05-24 15:19 UTC by Paweł Prażak
Modified: 2009-10-21 16:37 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Tabs bug screenshot (5.42 KB, image/png)
2009-05-24 15:20 UTC, Paweł Prażak
Details
Fixed corner widgets location in konqueror and close tab button vertical alignment (318.51 KB, image/png)
2009-10-20 09:16 UTC, Hugo Pereira Da Costa
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Paweł Prażak 2009-05-24 15:19:36 UTC
Version:           4.2.86 (KDE 4.2.86 (KDE 4.3 >= 20090514)) "release 126" (using 4.2.86 (KDE 4.2.86 (KDE 4.3 >= 20090514)) "release 126", KDE:KDE4:Factory:Desktop / openSUSE_11.1)
Compiler:          gcc
OS:                Linux (x86_64) release 2.6.27.21-0.1-default

Tabs in Koqueror are displayed improperly (regression).

The close button is displayed lower then it should be and it uses Qt's icon (rectangle one) instead of KDE's (round red one).

Open new tab and close tab tool buttons are displayed improperly, about 3px lower (see screen shot).

There is some kind of border between tab and tab content (about 3px) - see screen shot.

--
Linux 2.6.27.21-0.1-default x86_64
openSUSE 11.1 (x86_64)
Qt: 4.5.1
KDE: 4.2.86 (KDE 4.2.86 (KDE 4.3 >= 20090514)) "release 126"
Plasma Workspace: 0.3
x11-video-nvidiaG02-180.51-1.1
Comment 1 Paweł Prażak 2009-05-24 15:20:32 UTC
Created attachment 33972 [details]
Tabs bug screenshot
Comment 2 Dario Andres 2009-05-24 16:10:34 UTC
Have you tested with another widget styles ? (just to see if it is a Konqueror or an Oxygen issue)
Thanks
Comment 3 Paweł Prażak 2009-05-24 17:31:31 UTC
I'd surely do it, but unfortunately it's not working for me :|

See bug #193913

Any thoughts?
Comment 4 Paweł Prażak 2009-05-24 17:35:58 UTC
I've just remembered good old 'konqueror --style motif' :D
And yes, it looks like oxygen bug, because with motif it looks fine.

That would explain a lot, since I've checked the konquerors source code and haven't found any obvious tab related bugs.
Comment 5 Paweł Prażak 2009-05-24 17:44:34 UTC
I've should tried in other apps earlier, before assumption that it's konq's bug. It affects dolphin and kdevelop as well, but not the kmail(it has no close button on tabs).
Comment 6 Dario Andres 2009-05-24 17:57:21 UTC
Have you tested with --style plastique or --style cleanlooks ?
Comment 7 Paweł Prażak 2009-05-24 18:10:54 UTC
Yes, it works fine as well with plastique and cleanlooks.
Comment 8 Dario Andres 2009-05-24 18:25:55 UTC
Reassigning to oxygen
Comment 9 Paweł Prażak 2009-05-24 19:46:35 UTC
Thanks Dario :)
Comment 10 Huynh Huu Long 2009-05-24 23:06:30 UTC
> Open new tab and close tab tool buttons are displayed improperly, about 3px
lower (see screen shot).
We use PM_TabBarBaseOverlap, which doen't work well with Qt.

> There is some kind of border between tab and tab content (about 3px) - see
screen shot.
That's already fixed in trunk.

> Open new tab and close tab tool buttons are displayed improperly, about 3px
lower (see screen shot).
I don't know where this comes from, we don't do anything suspicious here, but I'll try to figure out...
Comment 11 Christoph Feck 2009-05-29 02:15:19 UTC
*** Bug 194377 has been marked as a duplicate of this bug. ***
Comment 12 Panagiotis Papadopoulos 2009-08-01 14:40:38 UTC
*** Bug 201903 has been marked as a duplicate of this bug. ***
Comment 13 Hugo Pereira Da Costa 2009-10-20 06:40:33 UTC
"The close button is displayed lower then it should be and it uses Qt's icon
(rectangle one) instead of KDE's (round red one)."

On the icon shape: this is a konqueror choice (as well as dophin)
On the icon position (with respect to the tab text). 
As far as I can tell, the issue comes from the fact that the contents margin of the tab are well accounted for to place the tab *text* but are ignored to place the tab button (it is always centered vertically).

Since the oxygen top and bottom margins are different (2 vs 4 pixmaps), this creates the observed misalignment.
Comment 14 Hugo Pereira Da Costa 2009-10-20 06:41:05 UTC
sorry. Typo: read (2 vs 4 pixels)
Comment 15 Hugo Pereira Da Costa 2009-10-20 07:22:52 UTC
sorry. Typo: read (2 vs 4 pixels)
Comment 16 Hugo Pereira Da Costa 2009-10-20 09:16:15 UTC
Created attachment 37677 [details]
Fixed corner widgets location in konqueror and close tab button vertical alignment
Comment 17 Hugo Pereira Da Costa 2009-10-20 09:17:16 UTC
Fixed in trunk (see previous attachment)
r1037814
r1037806
Comment 18 Hugo Pereira Da Costa 2009-10-21 16:37:36 UTC
SVN commit 1038638 by hpereiradacosta:

More fixing on tabwidget contents and tabbar layout.
Fixes conflict between r1038325 and r1037814 (and layout for konqueror, konversation, etc.)
CCBUG: 205094
CCBUG: 193893



 M  +12 -2     oxygen.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1038638