diff options
author | Maxim Monastirsky <momonasmon@gmail.com> | 2020-08-23 19:18:33 +0300 |
---|---|---|
committer | Maxim Monastirsky <momonasmon@gmail.com> | 2020-08-31 15:19:43 +0200 |
commit | 3e4968e6761c816c0a6aedf59485191af8a0fa4e (patch) | |
tree | e76c68df11b6f26ba5f27bb3e25e60eeb0880944 /officecfg | |
parent | 9bf4e2aa77c208d09b32c3638afab6f904bf0257 (diff) |
Remove remains of private:image/ via ImageIdentifier addon property
This is broken since commit 5c39b28a87060f80404079ab77604f664addb063
("tdf#96059 Replaced imageproducer with CommandInfoProvider") but so
far no one complained (maybe because the usefulness of such internal
images from extensions is questionable at least). Given also that
the whole ImageIdentifier feature (even its still working part) is
obsolete since OOo 2.0.3 (according to the OOo dev guide), and that
the availability of a particular image from an internal hardcoded
image list by a particular numerical id is more an implementation
detail, let's just remove the broken code instead of fixing it.
In the meantime, the code was also copied into the newly introduced
notebookbar addon code, so I handled it there too.
There are also the registry schema and a sdk example that mention this
feature, and need to be adjusted. Interesting that the particular
example used there - private:image/3216 is actually broken since 2011
with commit 2559cab126f81375197051fb5b07ba6abb9efc77
("FDO#42454 - EasyHack: remove code associated with unused icons").
Change-Id: I968b4fb8c5b207654476dd92c57d8db0815520ab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101529
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Diffstat (limited to 'officecfg')
-rw-r--r-- | officecfg/registry/schema/org/openoffice/Office/Addons.xcs | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/officecfg/registry/schema/org/openoffice/Office/Addons.xcs b/officecfg/registry/schema/org/openoffice/Office/Addons.xcs index 98d08a7b2f3a..98f4bf85fd63 100644 --- a/officecfg/registry/schema/org/openoffice/Office/Addons.xcs +++ b/officecfg/registry/schema/org/openoffice/Office/Addons.xcs @@ -38,7 +38,9 @@ </prop> <prop oor:name="ImageIdentifier" oor:type="xs:string"> <info> - <desc>A private URL used to reference an optional internal menu image. Example: private:image/3216 addresses the internal Office image with id=3216.</desc> + <desc>A base name for an image to be used for the given URL. _26.bmp and _16.bmp are + appended to that base name for large or small size icons respectively. Has higher + priority than images defined in the Images set.</desc> </info> </prop> <prop oor:name="Target" oor:type="xs:string"> @@ -168,7 +170,9 @@ </prop> <prop oor:name="ImageIdentifier" oor:type="xs:string"> <info> - <desc>A private URL used to reference an optional internal menu image. Example: private:image/3216 addresses the internal Office image with id=3216.</desc> + <desc>A base name for an image to be used for the given URL. _26.bmp and _16.bmp are + appended to that base name for large or small size icons respectively. Has higher + priority than images defined in the Images set.</desc> </info> </prop> <prop oor:name="Target" oor:type="xs:string"> @@ -351,7 +355,9 @@ </prop> <prop oor:name="ImageIdentifier" oor:type="xs:string"> <info> - <desc>A private URL used to reference an optional internal menu image. Example: private:image/3216 addresses the internal Office image with id=3216.</desc> + <desc>A base name for an image to be used for the given URL. _26.bmp and _16.bmp are + appended to that base name for large or small size icons respectively. Has higher + priority than images defined in the Images set.</desc> </info> </prop> <prop oor:name="Target" oor:type="xs:string"> |