-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123161/
-----------------------------------------------------------

(Updated März 28, 2015, 2:45 nachm.)


Review request for Plasma.


Changes
-------

Fixed pasting and implemented suggestions.


Bugs: 318221
    http://bugs.kde.org/show_bug.cgi?id=318221


Repository: kdeplasma-addons


Description
-------

The current displayed value from the textfield can be copied.
Content from the clipboard can be pasted but only if the content
is a valid number

BUG: 318221


Diffs (updated)
-----

  applets/calculator/package/contents/ui/calculator.qml 
23f74bd1ac7de6b7f4519677bd96aa351a91cff1 

Diff: https://git.reviewboard.kde.org/r/123161/diff/


Testing (updated)
-------

Copying:
*) Tried copying entered numbers and results - works for me :)

Pasting:
*) Pasting "asdf" leads to nothing being pasted - correct
*) Pasting "5" appends the number to the currently entered input or replaces 
the current number in case an operator has been pressed before

Only pasting doubles is possible - no signs or full math expressions are 
supported. 
It looks like the plasmoid doesn't supports signed numbers in general (e.g. 5 - 
-5 doesnt result in 10)

Comments welcome :) This is my second patch so hopefully my work isn't complete 
garbage :P


Thanks,

Bernhard Friedreich

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to