Page MenuHomePhabricator

Let file captions be edited on Mobile Frontend
Closed, ResolvedPublic

Assigned To
Authored By
Ramsey-WMF
Nov 7 2018, 8:16 PM
Referenced Files
F28159142: Image-1.jpg
Feb 8 2019, 8:08 PM
F28159647: recommended.png
Feb 8 2019, 8:08 PM
F27859821: mobile captions - edit mode.png
Jan 12 2019, 12:02 AM
F27859799: IMG_1852.png
Jan 12 2019, 12:02 AM
F27122921: image.png
Nov 8 2018, 7:19 PM
F27122942: image.png
Nov 8 2018, 7:19 PM
F27101428: Screenshot 2018-11-07 at 14.47.46.png
Nov 7 2018, 10:53 PM
F27098132: mobile-nocaption-edit.jpg
Nov 7 2018, 8:16 PM

Description

The mobile web view of Commons lacks an edit button to initiate editing of captions. See image below.

mobile-nocaption-edit.jpg (1×640 px, 60 KB)

Event Timeline

Change 472345 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/WikibaseMediaInfo@master] WIP: Mobile editing of captions, sort-of

https://gerrit.wikimedia.org/r/472345

With some hacking around in the Wikibase code, I got the code executing and it works fine (but looks ugly as sin):

Screenshot 2018-11-07 at 14.47.46.png (1×2 px, 166 KB)

Happy to do a mobile-specific set of styling, but first we need to determine with the Wikibase team whether this approach is OK by them.

Some quick styling changes gets us to:

Read modeEdit mode
image.png (425×448 px, 40 KB)
image.png (607×445 px, 50 KB)

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

This is good enough for v1, I think :)

You mentioned earlier wanting to check in with Team Wikibase on these changes. Have you initiated that already or shall I?

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

This is good enough for v1, I think :)

You mentioned earlier wanting to check in with Team Wikibase on these changes. Have you initiated that already or shall I?

I'll do it.

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

@Jdforrester-WMF will T204934 fix that?

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

@Jdforrester-WMF will T204934 fix that?

Yes. Something to work on once we're in prod. :-)

Proper mobile design is not possible given how the items are rendered on the page right now, sadly. :-(

@Jdforrester-WMF will T204934 fix that?

Yes. Something to work on once we're in prod. :-)

Specifically that looks like implementation of T201463.

Jdforrester-WMF renamed this task from Beta Commons: File Captions can't be edited on mobile browsers to Let file captions be edited on Mobile Frontend.Jan 11 2019, 9:52 PM
Jdforrester-WMF updated the task description. (Show Details)

The Structured data header is also missing on mobile:

IMG_1852.png (2×1 px, 1 MB)

Related to the styling changes above, if mobile web follows the Upload Wizard pattern, we'll get more width for the caption text:

mobile captions - edit mode.png (436×320 px, 30 KB)

Change 472344 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/Wikibase@master] Enable a few lib and repo RL modules in mobile target for SDC

https://gerrit.wikimedia.org/r/472344

Change 472344 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Enable a few lib and repo RL modules in mobile target for SDC

https://gerrit.wikimedia.org/r/472344

Change 472345 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Mobile editing of captions

https://gerrit.wikimedia.org/r/472345

This will go live on 2019-02-05 on TestCommons and 2019-02-06 on Commons.

This isn't working on TestCommons for some reason. Am digging in.

Ignore that, TestCommons flipped back to wmf.14 without me noticing. All looks good.

Tested but not working properly.

Bugs found when editing while not logged in (both on Safari and Chrome on iOS):

  1. Attempts to add a NEW English Caption results in "Revision with ID not found" error. Editing existing captions seems to work fine for some files (on this image, editing a caption worked)
  2. Additionally, on the link above, the English caption I added via the desktop web interface ended up showing twice on the mobile interface after page refresh.
  3. Also, the entire caption section appears TWICE on the page; once in the correct place, but again just under the "Original Upload Log" section.
  4. On some files, existing captions do not appear at all on the mobile interface (this file as an example, which has both an existing English and German caption but neither appear in the mobile interface)

@Jdforrester-WMF @Ramsey-WMF

I have some feedback I'd like you to consider that will make the Captions mobile experience better:

Image-1.jpg (2×1 px, 290 KB)
recommended.png (736×414 px, 125 KB)
current implementationrecommended design

Issues with current implementation:

  • Trash can is awkwardly placed
  • There is no gap between the language dropdown and the caption field, making it look like they run into each other
  • The width of the caption field is too short and the help text is impossible to read

Recommended changes

  • Move trash can to the same line as the language dropdown (just like Upload Wizard)
  • There needs to be a gap between the language dropdown and the caption field
  • Make the width of the caption field longer. Could go as far as to the end of “Publish changes”

I agree with these recommendations.

Erm ... guys, if patches for this ticket have been merged and are now live (which they have and they are), then IMO it's better to create new tickets for any outstanding issues rather than keep adding to this ticket