Review Request 124374: custom inline edit menu for plasma controls

Marco Martin notmart at gmail.com
Wed Sep 2 09:44:59 UTC 2015



> On Sept. 2, 2015, 9:15 a.m., Kai Uwe Broulik wrote:
> > src/declarativeimports/plasmastyle/EditMenuTouch.qml, line 62
> > <https://git.reviewboard.kde.org/r/124374/diff/3/?file=399938#file399938line62>
> >
> >     Imho it's cleaner if we just have a floating paste button if you cannot copy/paste but that's up to the designers I guess, though a word will always be selected when we bring up the menu? There's no "tap, wait, paste" feature?

a simple tap just moves the cursor and opens the menu, without selecting.
a long press selects the word


- Marco


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


On Sept. 1, 2015, 12:14 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124374/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2015, 12:14 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This depends from https://codereview.qt-project.org/#/c/121659/
> provides the needed elements for touch based text selection and inline (no separate window) menu for cut/copy/paste
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/plasmastyle/CursorDelegate.qml PRE-CREATION 
>   src/declarativeimports/plasmastyle/CursorHandleStyle.qml PRE-CREATION 
>   src/declarativeimports/plasmastyle/EditMenuTouch.qml PRE-CREATION 
>   src/declarativeimports/plasmastyle/SelectionHandleStyle.qml PRE-CREATION 
>   src/declarativeimports/plasmastyle/TextAreaStyle.qml 114c322 
>   src/declarativeimports/plasmastyle/TextFieldStyle.qml 635c938 
> 
> Diff: https://git.reviewboard.kde.org/r/124374/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> snapshot1.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/07/16/7fb4e7da-e705-4bd1-b084-1a69145a0bc2__snapshot1.png
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150902/6bce6edb/attachment.html>


More information about the Plasma-devel mailing list