[Libreoffice] [PATCH] set the icons size based on the DPI just like we do on Mac
michael.meeks at novell.com
Wed Jan 5 06:27:58 PST 2011
On Mon, 2011-01-03 at 02:16 +0100, Christian Lohmaier wrote:
> > - aStyleSet.SetToolbarIconSize( STYLE_TOOLBAR_ICONSIZE_LARGE );
> > + aStyleSet.SetToolbarIconSize( nDispDPIY > 160 ? STYLE_TOOLBAR_ICONSIZE_LARGE
> : STYLE_TOOLBAR_ICONSIZE_SMALL );
On large numbers of machines, nDispDPI is just broken - it is fetched
(AFAIR) from Xft - and is ~hard-coded to 75 everywhere to save lots of
grief from X servers getting it horribly wrong.
So - the net effect of this is to simply default to tiny icons, even on
> How much effort would it be to read the property from gconf if
> available (or is this already done when DESKTOP=gnome)?
That shouldn't be necessary - I believe we shoudl sync the Xft setting;
but perhaps there is some xsettings linkage we could add: gtk-xft-dpi or
> I prefer small icons, no matter what. Icons are there to preserve
> space, provide access to functionality, not to waste my screen-estate
> with "pretty pictures".
Sure - but you know what the icons do :-) and you are expert with the
keybindings. Of course - if you are an expert, you can turn the icons to
a smaller size too ;-) Personally, my emacs has no toolbar, or menu bar
but ... this is not a good default.
So - I would prefer to stick with large icons; and not use the DPI
setting. I suggest instead, that we only use large icons if the true Y
resolution > 768 - how does that sound ?
michael.meeks at novell.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice