To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=112102
------- Additional comments from [email protected] Tue Jun 15 09:47:09 +0000 2010 ------- If we use QStyle::PM_LayoutLeftMargin, we will not see the status bar in most KDE4 themes. It is too wide and it replaces the useful content there. Hmm, I see the border in the new Writer window with the Oxygen theme with my diff. I think that the important difference is the "+1" in int nFrameWidth = kapp->style()->pixelMetric(QStyle::PM_DefaultFrameWidth) + 1; I do not like magic constants. A better solution might be to get the real border width via QFrame:frameWidth(), see http://doc.qt.nokia.com/4.6/qframe.html#frameWidth-prop. Though, I am not mistaken, we do not have the real QT widget available in the method. The "QStyle::PM_DefaultFrameWidth) + 1" looks like a good compromise. It works well in many themes. In each case, I think that the content is more important than the border ;-) --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
