Review Request 124246: WIP: Improve sticky notes
Kai Uwe Broulik
kde at privat.broulik.de
Fri Jul 3 21:13:10 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124246/
-----------------------------------------------------------
Review request for Plasma and KDE Usability.
Repository: kdeplasma-addons
Description
-------
This patch adds the following improvements to the sticky notes applet:
- A config dialog with previews of the individual colors, the ability to change the font size, and font color (once the KDeclarative ColorButton is merged, see Review 124014) - are there any other important options the 4.x one had?
- Ctrl+Mousewheel to change font size, I always used this "feature" in 4.x and was disappointed that it would not actually save the font size
- Focus the input field when expanding the applet (eg. when it is in a panel, click the note, start typing)
Diffs
-----
applets/notes/package/contents/config/config.qml PRE-CREATION
applets/notes/package/contents/config/main.xml 596dfc0
applets/notes/package/contents/ui/configAppearance.qml PRE-CREATION
applets/notes/package/contents/ui/main.qml 1f1022c
applets/notes/plugin/documenthandler.h 41cf704
applets/notes/plugin/documenthandler.cpp 79c276e
Diff: https://git.reviewboard.kde.org/r/124246/diff/
Testing
-------
Changing the font size from the config dialog and mousewheeling seems to work without screwing up your existing formatting.
Unfortunately the config dialog is quite difficult to get to since right clicking the text area yields the standard copy/paste menu and I failed to manually add another item for the applet config there
File Attachments
----------------
Config dialog
https://git.reviewboard.kde.org/media/uploaded/files/2015/07/03/463ae591-0201-4e52-ac63-f3083790da04__stickylove2.png
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150703/a7b69c61/attachment.html>
More information about the Plasma-devel
mailing list