Issue 22098 - ugly rulers with larger font sizes ...
Summary: ugly rulers with larger font sizes ...
Status: CLOSED FIXED
Alias: None
Product: gsl
Classification: Code
Component: code (show other issues)
Version: OOo 1.1 RC5
Hardware: PC Linux, all
: P4 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: ru
QA Contact: issues@gsl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-04 11:05 UTC by mmeeks
Modified: 2010-11-10 16:48 UTC (History)
1 user (show)

See Also:
Issue Type: ENHANCEMENT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description mmeeks 2003-11-04 11:05:10 UTC 
When one has a font size of ~14points or so - which looks reasonable for menus
(on my display), the ruler starts to look slightly daft; this patch makes it
look slightly less so ;-)

--- vcl/unx/source/gdi/dtint.cxx	28 May 2003 12:33:47 -0000	1.16
+++ vcl/unx/source/gdi/dtint.cxx	4 Nov 2003 10:57:57 -0000
@@ -573,7 +564,6 @@ void DtIntegrator::GetSystemLook( const 
                     aStyleSettings.SetAppFont( aFont );
                     aStyleSettings.SetHelpFont( aFont );
                     aStyleSettings.SetMenuFont( aFont );
-                    aStyleSettings.SetToolFont( aFont );
                     aStyleSettings.SetLabelFont( aFont );
                     aStyleSettings.SetInfoFont( aFont );
                     aStyleSettings.SetRadioCheckFont( aFont );
@@ -581,6 +571,13 @@ void DtIntegrator::GetSystemLook( const 
                     aStyleSettings.SetFieldFont( aFont );
                     aStyleSettings.SetIconFont( aFont );
                     aStyleSettings.SetGroupFont( aFont );
+
+					// We want the tool font to be slightly smaller,
+					// otherwise the ruler gets horribly bloated
+					long nHeight = aFont.GetHeight();
+					nHeight = ( nHeight + 8 ) / 2;
+					aFont.SetHeight( nHeight );
+                    aStyleSettings.SetToolFont( aFont );
                 }
                 break;
             case ToolbarIconSize:
Comment 1 christof.pintaske 2003-11-05 17:40:49 UTC 
cp->ssa: I'm not quite satisfied with the patch. If the ruler does
look bad with the current font size settings this should be taken care
for in a platform independent manner.

Wouldn't this patch be counterproductive for accessibility ?

Please have a look at it.
Comment 2 stephan_schaefer 2003-11-24 11:09:38 UTC 
I will look at it once i16682 is done. And of course it will not be
done UNIX only.
Comment 3 stephan_schaefer 2003-11-26 14:03:39 UTC 
Fixed in CWS vcl17. The change now affects all platforms.
Comment 4 stephan_schaefer 2004-01-23 11:59:28 UTC 
ssa->ru: please verify in CWS vcl17.
Comment 5 ru 2004-01-27 16:14:23 UTC 
.
Comment 6 ru 2004-01-27 16:14:59 UTC 
.
Comment 7 ru 2004-04-28 14:47:10 UTC 
.
Comment 8 inlawcada 2010-11-10 16:48:12 UTC 
Created attachment 73291