[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