----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123161/#review78129 -----------------------------------------------------------
applets/calculator/package/contents/ui/calculator.qml (line 95) <https://git.reviewboard.kde.org/r/123161/#comment53528> avoid guessing what the shortcut is. Otherwise if someone maps it to alt in their settings it won't work. Or if someone has a dvorak layout it won't be c and v. Use if event.matches(StandardKey.Copy) http://doc.qt.io/qt-5/qml-qtquick-keyevent.html#matches-method internally this will ask our QPT plugin which loads things from a KDE config file. applets/calculator/package/contents/ui/calculator.qml (line 220) <https://git.reviewboard.kde.org/r/123161/#comment53531> I'm not convinced this is going to work. You'll update the display, but all the handling of storing what number is entered handles in the key press events. Can you try pasting "5" then typing '+' and '1' and see what happens. I think it will then show 1, not 6. - David Edmundson On March 27, 2015, 11:21 p.m., Bernhard Friedreich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123161/ > ----------------------------------------------------------- > > (Updated March 27, 2015, 11:21 p.m.) > > > Review request for Plasma. > > > 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 > ----- > > applets/calculator/package/contents/ui/calculator.qml > 23f74bd1ac7de6b7f4519677bd96aa351a91cff1 > > Diff: https://git.reviewboard.kde.org/r/123161/diff/ > > > Testing > ------- > > Copying: > *) Tried copying entered numbers and results - works for me :) > > Pasting: > *) Pasting "asdf" leads to nothing being pasted - correct > *) Pasting "5" pasts 5 - replaces the currently entered text with the pasted > content (this could be worked around - should it append?) > *) Pasting "= 5 + 3" passes through validation - no idea why yet - this > should be possible I guess? > > 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
