So my GNOME settings are by default to not show icons in menus so I
didn't notice that we show a placeholder graphic for all missing icons
in menus.
That means though that *all* menu entries have an icon beside them when
icons are turned on, and a load of them are the broken graphic
placeholder.
Where are we on the thinking/plan here...
Do we think that every entry *should* have an icon and that each missing
one is a bug and needs to be drawn.
Or do we think that it's ok to have no menu entry icon and should just
draw no placeholder, e.g. something like the attached patch ? when they
are missing.
C.
diff --git a/framework/source/uiconfiguration/imagemanagerimpl.cxx
b/framework/source/uiconfiguration/imagemanagerimpl.cxx
index bca1f2d..bdd0347 100644
--- a/framework/source/uiconfiguration/imagemanagerimpl.cxx
+++ b/framework/source/uiconfiguration/imagemanagerimpl.cxx
@@ -363,15 +363,7 @@ GlobalImageList::~GlobalImageList()
Image GlobalImageList::getImageFromCommandURL( sal_Int16 nImageType, const rtl::OUString&
rCommandURL )
{
osl::MutexGuard guard( getGlobalImageListMutex() );
- Image aImage = CmdImageList::getImageFromCommandURL( nImageType, rCommandURL );
- if (!aImage)
- {
- BitmapEx rBitmap;
- bool res = ::vcl::ImageRepository::loadDefaultImage(rBitmap);
- if (res)
- aImage = Image(rBitmap);
- }
- return aImage;
+ return CmdImageList::getImageFromCommandURL( nImageType, rCommandURL );
}
bool GlobalImageList::hasImage( sal_Int16 nImageType, const rtl::OUString& rCommandURL )
Context
- [Libreoffice] lots of fallback icons shown in menus · Caolán McNamara
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.