Bug 73592 - lower titlebar border missing in KDE 1 Window Decoration
Summary: lower titlebar border missing in KDE 1 Window Decoration
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: decorations (show other bugs)
Version: unspecified
Platform: Slackware Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-27 11:56 UTC by sombragris
Modified: 2004-08-03 19:49 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description sombragris 2004-01-27 11:56:30 UTC
Version:            (using KDE KDE 3.2.0)
Installed from:    Slackware Packages
OS:          Linux

Hi,

Thanks for all the work in KDE 3.2. You might remember me as the one from bug report 71552 <http://bugs.kde.org/show_bug.cgi?id=71552>, where I reported that the KDE 1 windec was missing. In a move that I really appreciate, you fixed this.

Now I have a problem to report: The lower part of the titlebar is not there and the window contents clip into the titlebar as a result. It looks extremely ugly.

The best way to appreciate this is to see an image in a Kuickshow window. You'll see the upper part of the image clipping into the lower part of the titlebar.

I request that this issue be fixed, if possible. I will really appreciate it.

Thanks,


Eduardo Sanchez
Asuncion, Paraguay, South America
Member of the KDE-ES translation team
Comment 1 Lubos Lunak 2004-01-30 18:23:26 UTC
Happens with small titlebar font sizes.
Comment 2 sombragris 2004-05-10 20:45:59 UTC
Same behavior in KDE 3.2.2. I respectfully request a fix.

Thanks,

Eduardo Sanchez
KDE-ES Spanish translation team
Comment 3 Sandro Giessl 2004-08-03 19:42:23 UTC
CVS commit by giessl: 

Fix titlebar height, sorry for the long delay.
CCMAIL: 73592-done@bugs.kde.org


  M +1 -1      kde1client.cpp   1.36


--- kdeartwork/kwin-styles/kde1/kde1client.cpp  #1.35:1.36
@@ -521,5 +521,5 @@ void StdClient::borders( int& left, int&
 {
     left = right = bottom = 6;
-    top = 6 + widget()->fontMetrics().lineSpacing();
+    top = 6 + widget()->fontMetrics().lineSpacing()+2;
 }
 


Comment 4 sombragris 2004-08-03 19:49:41 UTC
Sandro, Thank you so much!! I can't even begin to tell you how much I missed that windec.

Eduardo