[Differential] [Request, 34 lines] D3967: [Fifteen Puzzle] Add "Browse" button
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Wed Jan 4 18:18:47 UTC 2017
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R114 Plasma Addons.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
There's a text field for entering a path to an image to use as puzzle. Add a Browse button to make it actually useful.
We'll strip the file:// prefix as the image handler cannot cope with remote images anyway.
TEST PLAN
The text field isn't disabled when the checkbox is unchecked, instead, it's smarter: as soon as you start typing in the text field or click the browse button, it's automatically checked. Saves you a enable checkbox and then press browse dance. (We should use this approach more imho)
REPOSITORY
R114 Plasma Addons
REVISION DETAIL
https://phabricator.kde.org/D3967
AFFECTED FILES
applets/fifteenPuzzle/package/contents/ui/configAppearance.qml
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170104/3d6854b4/attachment-0001.html>
More information about the Plasma-devel
mailing list