[Libreoffice-bugs] [Bug 129879] Qt UI swapped button order in save dialog

bugzilla-daemon at bugs.documentfoundation.org bugzilla-daemon at bugs.documentfoundation.org
Wed Jan 8 10:42:44 UTC 2020


https://bugs.documentfoundation.org/show_bug.cgi?id=129879

--- Comment #3 from Fabian Vogt <fabian at ritter-vogt.de> ---
There seems to be a mismatch between what Application::GetDesktopEnvironment()
returns and what getButtonPriority() in layout.cxx expects. This should fix it:

diff --git a/vcl/source/window/layout.cxx b/vcl/source/window/layout.cxx
index 981ac62..9c5a363 100644
--- a/vcl/source/window/layout.cxx
+++ b/vcl/source/window/layout.cxx
@@ -718,8 +718,8 @@ static int getButtonPriority(const OString &rType)
     const OUString &rEnv = Application::GetDesktopEnvironment();

     if (rEnv.equalsIgnoreAsciiCase("windows") ||
-        rEnv.equalsIgnoreAsciiCase("tde") ||
-        rEnv.startsWithIgnoreAsciiCase("kde"))
+        rEnv.equalsIgnoreAsciiCase("lxqt") ||
+        rEnv.startsWithIgnoreAsciiCase("plasma"))
     {
         pOrder = &aSaveDiscardCancel[0];
     }

LXQt uses Save/Discard/Cancel as well, so I added that and removed tde, which
is not returned anymore.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200108/058c7910/attachment-0001.htm>


More information about the Libreoffice-bugs mailing list