diff options
author | Justin Luth <justin_luth@sil.org> | 2020-02-06 14:54:19 +0300 |
---|---|---|
committer | Maxim Monastirsky <momonasmon@gmail.com> | 2020-02-06 23:11:39 +0100 |
commit | 87a8b2c0d1002bff4f6268ac8673dd32fc7762b9 (patch) | |
tree | 1fdbef6ba9f2b9e8dabee2b305c2595b5c547eba /configmgr/qa/unit | |
parent | f3f5e11682b0392b4e750ec014c399051a5fe27c (diff) |
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_luth@sil.org>
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Tested-by: Maxim Monastirsky <momonasmon@gmail.com>
Diffstat (limited to 'configmgr/qa/unit')
-rw-r--r-- | configmgr/qa/unit/test.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
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() { |