Skip to content
Commit 435d2ad1 authored by Nate Graham's avatar Nate Graham
Browse files

Give Exit Full Screen and Configure buttons text and adjust wording

Summary:
Unlike Kate, Gwenview's {nav Exit Full Screen} button is icons-only, impeding
discoverability. This patch shows its action's text, and adjusts that text to be more
consistent with the typical wording.

In the process, the button below it is also given visible text to avoid making it look
odd in its layout.

To accomplish this technically, the `FullScreenToolBar` class's `addAction()` function
is given a new optional parameter for specifying if and where the button's text should
appear, which is then used to give the aforementioned buttons visible text according to
the user's toolbutton text display preferences. The actions are also no longer marked
with the Low Priority status, which forces them to be always icons-only and was
previously redundant since icons-only status was forced in a different way. Fixed now.

BUG: 385314
FIXED-IN: 20.04.0

Test Plan: {F8176526}

Reviewers: #gwenview, #vdg, broulik

Reviewed By: broulik

Tags: #gwenview

Differential Revision: https://phabricator.kde.org/D28052
parent cf4ce008
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment