Re: [KDE Usability] Review Request 128413: Add clipboard buttons in the Checksums tab

Elvis Angelaccio <[email protected]>
Newsgroups gmane.comp.kde.usability,gmane.comp.kde.devel.frameworks
Message-ID <[email protected]>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128413/
-----------------------------------------------------------

(Updated July 22, 2016, 9:21 p.m.)


Review request for KDE Frameworks, KDE Usability and David Faure.


Changes
-------

Patch rebased.

@dfaure: is this ok code-wise?


Repository: kio


Description
-------

As discussed in RR 128283, we can improve the usability of the dialog if we add buttons to interact with the clipboard.

The Paste button is always visible, while the three Copy buttons are hidden until the correspoding checksum is computed.


Diffs (updated)
-----

  src/widgets/checksumswidget.ui 543088e 
  src/widgets/kpropertiesdialog.cpp f6d835e 

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


Testing
-------

Copy/Paste works as expected.


File Attachments (updated)
----------------

Paste button next to the input field.
  https://git.reviewboard.kde.org/media/uploaded/files/2016/07/22/0b0d3f06-7294-4ff1-b59d-d13c413c00f5__Spectacle.C31997.png
Example of Copy button
  https://git.reviewboard.kde.org/media/uploaded/files/2016/07/22/5f137d47-4592-4f4c-be96-4c2fc3d09978__Spectacle.J32017.png


Thanks,

Elvis Angelaccio

_______________________________________________
kde-usability mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-usability
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.