Review Request 117041: kdeplasma-addons/applets/pastebin: add two new options
Gregor Mi
codestruct at posteo.org
Sat Apr 4 16:11:07 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117041/
-----------------------------------------------------------
(Updated April 4, 2015, 4:11 p.m.)
Status
------
This change has been discarded.
Review request for Plasma, Aleix Pol Gonzalez, Dominik Haumann, and Sebastian Kügler.
Repository: kdeplasma-addons
Description
-------
Summary
-------
Add two options:
1. Show confirmation before sending (to prevent accidental sending of maybe sensible data)
2. Appearance: Restrict width on large host panel (to prevent huge icon on large panels)
How does this change manifest on screen?
----------------------------------------
Settings dialog:
- before change: http://wstaw.org/m/2014/03/25/plasma-desktopXL1751.png
- after change: http://wstaw.org/m/2014/03/25/screenshot_default.png
Behaviour:
1. "Show confirmation before sending" turned on and after dropping an image: http://wstaw.org/m/2014/03/25/plasma-desktopGd1751.png
2. "Appearance: Restrict width on large host panel"
- turned off (default): http://wstaw.org/m/2014/03/25/plasma-desktopOt1751.png (pretty big compared to other icons in system tray)
- turned on (new): http://wstaw.org/m/2014/03/25/plasma-desktopOL1751.png (the width is restricted so it does not eat up horizontal space)
Open issues
-----------
- Is there a default "review board TARGET_GROUP" for kdeplasma-addons review-request?
@apol: I saw that in the frameworks branch you replaced pastebin with quickshare. I haven't had the chance to test it out. Does it already have the same features as pastebin?
Diffs
-----
applets/pastebin/pastebinConfig.ui 540fae733c0ea738b48b3bf2b11b9f9b10bdb453
applets/pastebin/pastebin.cpp df7f534c17d5a383480f385b83948d01635641a7
applets/pastebin/pastebin.h 931057f9df4f5e875f3984df1f40869235d8297e
.gitignore PRE-CREATION
.reviewboardrc PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/117041/diff/
Testing
-------
See screenshots in description. Test if settings are getting stored. Test with different form factors (horizontal, vertical).
Thanks,
Gregor Mi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150404/b5765ebd/attachment-0001.html>
More information about the Plasma-devel
mailing list