| MingleTerminator | |
| Dec 8 2014, 4:56 PM |
| F5144066: pasted_file | |
| Dec 23 2016, 4:51 PM |
| F5141901: embed-aligned.png | |
| Dec 23 2016, 12:09 PM |
| F5141907: ebed-aligned-lines.png | |
| Dec 23 2016, 12:09 PM |
| F5134601: pasted_file | |
| Dec 22 2016, 6:14 PM |
Migrated from: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/366
As a user,I want to copy code with a single click, so that I can share/reuse it comfortably
Very low priority, pre-selecting the text gives us an only slightly more inconvenient method of copying.
Needs an icon. Could use the one from clippy - doesn't really match the Wikimedia design but used by a lot of apps (e.g. gerrit, github) so easy to recognize.
For ContentTranslation we used this paste icon which may work.
@violetto may want to review it (it's under the "Not final" icon folder).
Mass-removing the Multimedia tag from MediaViewer tasks, as this is now being worked on by the Reading department, not Editing's Multimedia team.
Gilles pointed out https://hacks.mozilla.org/2015/09/flash-free-clipboard-for-the-web/ and http://caniuse.com/#feat=clipboard according to which pure-JS copy is now a viable option.
Where are those share/embed fields are located ? Can you send me a page where those fields exist ?
They are visible on any image. See
https://www.mediawiki.org/wiki/Help:Extension:Media_Viewer#How_can_I_share_an_image.3F
and the answer after that.
I will mentor this in Google-Code-In-2016
Seems simple enough for me to introduce the student to image viewer, while allowing me to go understand it better as well.
Looking forward to it.
Change 328696 had a related patch set uploaded (by Sn1per):
Add button to copy share/embed text to clipboard
What it looks like now:
Especially for the embed view, there are some small layout aspects you may consider. The icon placement makes it a bit difficult to visually group the elements consistently. I'd propose to adjust the spacing and the size of the icon so that the different groups (code box and settings) are more clear:
I included also a version with the guides I used as a reference (blue line is about 10px while the orange is 20px). These are intended to help visually grouping the code box with the annotation and copy action below it as one main element and the settings as another (the separation between the setting controls seem to be lost as part of a previous regression).
@Pginer-WMF what do you think, can we remove the select-on-click behavior after this and improve T110579, or is the copy icon not discoverable enough for that?
The selection of the whole text seems to align with the main behaviour, so I'd not remove it.
It would be worth analysing whether we need to support selecting parts of the text (do we know of examples where this has been a limitation?).
If we need to support partial selection, it can be done in a similar way as major browsers do with the URL box: initial click when the input box gets focused selects the whole text but further clicks allow for regular text manipulation (in our case, that would be just selection of some parts, not modifying the text). This approach emphasises the main usecase of copying the whole thing using the well-known keyboard shortcut, but allows for more advanced usecases of extracting parts when needed.
Change 328696 merged by jenkins-bot:
Add button to select and copy share/embed text to clipboard
People have complained about it multiple times; I think it's not so much about partial selection than the ability to read the text before copying it.
If we need to support partial selection, it can be done in a similar way as major browsers do with the URL box: initial click when the input box gets focused selects the whole text but further clicks allow for regular text manipulation (in our case, that would be just selection of some parts, not modifying the text). This approach emphasises the main usecase of copying the whole thing using the well-known keyboard shortcut, but allows for more advanced usecases of extracting parts when needed.
Yeah, I tried that at some point in https://gerrit.wikimedia.org/r/#/c/178219/ but it turned out to be more complicated than what I had time for (due to some sort of browser difference in event handling).
There is a boilerplate text field on download as well (after clicking on the show attribution footer); I don't think there is any reason not to add the copy button there. I guess the task was created before that text field existed and then we forgot to update it.
Change 329495 had a related patch set uploaded (by Sn1per):
Copy button for attribution text in download dialog
Change 329495 merged by jenkins-bot:
Copy button for attribution text in download dialog