Review Request 122402: [clipboard] Show barcodes directly in view

Martin Gräßlin mgraesslin at kde.org
Wed Feb 4 12:18:39 UTC 2015



> On Feb. 4, 2015, 12:05 p.m., Martin Klapetek wrote:
> > applets/clipboard/contents/ui/BarcodePage.qml, lines 103-106
> > <https://git.reviewboard.kde.org/r/122402/diff/5/?file=346784#file346784line103>
> >
> >     Why not switch the logic and put the show() inside the if? I mean the return in basically one-line function seems quite redundant...
> >     
> >     
> >     ```
> >     if (barcodeView.visible) {
> >         barcodeView.show(barcodeView.uuid);
> >     }
> >     ```

that can be simplified even further: the complete check is no longer needed with the switch to PageStack. (Testing and uploading new version as I write).


- Martin


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


On Feb. 4, 2015, 11:21 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122402/
> -----------------------------------------------------------
> 
> (Updated Feb. 4, 2015, 11:21 a.m.)
> 
> 
> Review request for Plasma and Sune Vuorela.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> When clicking the barcode button, the view is changed to just show
> the barcode instead of opening a dialog. By default a QR Code is
> displayed.
> 
> A tool button is added which allows to change the barcode type, with
> the following barcodes being supported:
> * QR
> * Data Matrix
> * Code 39
> * Code 93
> 
> 
> Diffs
> -----
> 
>   applets/clipboard/contents/ui/ClipboardPage.qml PRE-CREATION 
>   applets/clipboard/contents/ui/clipboard.qml 23cabadd2fec73b2c5343439b821e3d819223eeb 
>   klipper/CMakeLists.txt a08f062480b15f32f049e2d0d0e311dbe2964c02 
>   klipper/clipboardjob.cpp ad6b6358d0d9bfce25045f997f017e262d6e7f53 
>   klipper/org.kde.plasma.clipboard.operations 5e9f71ac1888fbce90f8c6e55decaf8e737bea0d 
>   applets/clipboard/contents/ui/BarcodePage.qml PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/122402/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> QR Code
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/02/03/ba143497-eb97-4389-93ca-752e309386c1__missing-tasks-applet1.png
> data Matrix
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/02/03/5dde29fb-7044-453a-9544-f8f6b9ba5cef__missing-tasks-applet1.png
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150204/652c4be1/attachment.html>


More information about the Plasma-devel mailing list