configmgr/qa/unit/test.cxx | 2 +-
officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu | 8
++++----
2 files changed, 5 insertions(+), 5 deletions(-)
New commits:
commit 87a8b2c0d1002bff4f6268ac8673dd32fc7762b9
Author: Justin Luth <justin_l...@sil.org>
AuthorDate: Thu Feb 6 14:54:19 2020 +0300
Commit: Maxim Monastirsky <momonas...@gmail.com>
CommitDate: Thu Feb 6 23:11:39 2020 +0100
GenericCommands: fix Fontwork tooltips
These were terribly confusing.
.uno:Fontwork does NOT insert fontwork text, despite the tooltip name.
It opens up the old fontwork dialog that modifies contour properties
so that the text can flow along the contour.
The UIname is Fontwork, so lets have an automatic tooltip for that.
.uno:FontworkGalleryFloater has a UIname of "Fontwork Style",
but the auto-generated tooltip was "Fontwork". (Notice that this
conflicts with the UIname of the other one. So when you add "Fontwork"
to your toolbar, and chose the icon with the tooltip "Fontwork" you
get a different item.) Since this one actually inserts text, and
doesn't actually modify any existing style (that's the function of
Fontwork Shape), lets give this one the tooltip "Insert Fontwork Text".
Therefore, this patch effectively switches the two tooltip names around,
which much better matches their function, and their UIname.
I expect that this was the original 2016 intention anyway...
At the end, I decided to also change the UIname to "Insert Fontwork",
since the gallery is normally added to insert menus...
The unit test I had to modify just looked like a functionality test
and had nothing to do specifically with the label's string.
(It was last changed for tdf#91781 which made no specific
reference to fontwork.)
This might help documentation bug tdf#118336 a bit too.
Change-Id: I152596781def2d8dba47f53e345976543e3131bc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88101
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_l...@sil.org>
Reviewed-by: Maxim Monastirsky <momonas...@gmail.com>
Tested-by: Maxim Monastirsky <momonas...@gmail.com>
diff --git a/configmgr/qa/unit/test.cxx b/configmgr/qa/unit/test.cxx
index 6d98f36154ef..e553c1947f66 100644
--- a/configmgr/qa/unit/test.cxx
+++ b/configmgr/qa/unit/test.cxx
@@ -272,7 +272,7 @@ void Test::testSetSetMemberName()
".uno:FontworkShapeType",
"Label") >>=
s);
- CPPUNIT_ASSERT_EQUAL( OUString("Fontwork Style"), s );
+ CPPUNIT_ASSERT_EQUAL( OUString("Insert Fontwork"), s );
}
void Test::testInsertSetMember() {
diff --git
a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
index 87bd85857eaf..c266d12afbfe 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
@@ -72,11 +72,14 @@
</node>
<node oor:name=".uno:FontworkGalleryFloater" oor:op="replace">
<prop oor:name="Label" oor:type="xs:string">
- <value xml:lang="en-US">Fontwork Style</value>
+ <value xml:lang="en-US">Insert Fontwork</value>
</prop>
<prop oor:name="ContextLabel" oor:type="xs:string">
<value xml:lang="en-US">Fontwork...</value>
</prop>
+ <prop oor:name="TooltipLabel" oor:type="xs:string">
+ <value xml:lang="en-US">Insert Fontwork Text</value>
+ </prop>
<prop oor:name="Properties" oor:type="xs:int">
<value>9</value>
</prop>
@@ -3848,9 +3851,6 @@
<prop oor:name="Label" oor:type="xs:string">
<value xml:lang="en-US">F~ontwork</value>
</prop>
- <prop oor:name="TooltipLabel" oor:type="xs:string">
- <value xml:lang="en-US">Insert Fontwork Text</value>
- </prop>
<prop oor:name="Properties" oor:type="xs:int">
<value>1</value>
</prop>
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
