Page MenuHomePhabricator

[L] Add Review step for Alt text experiment and update publish
Closed, ResolvedPublic5 Estimated Story Points

Assigned To
None
Authored By
HNordeenWMF
Jul 17 2024, 8:22 PM
Referenced Files
F57282665: Screenshot 2024-08-20 at 8.50.20 AM.jpeg
Aug 20 2024, 1:53 PM
F57282517: IMG_7662 copy.PNG
Aug 20 2024, 10:23 AM
F57282515: image.png
Aug 20 2024, 10:23 AM
F57282512: IMG_7662.PNG
Aug 20 2024, 10:23 AM
F57281246: IMG_7655.png
Aug 19 2024, 11:53 AM
F56488763: image.png
Jul 17 2024, 8:22 PM

Description

Background

After a user writes alt text, they should have the opportunity to review what they have written before publishing.

Requirements
  • Image, Alt text entered, and image caption (if available) are shown for user to review before publishing
  • "Back" brings them back to the active input box, and text already typed is retained
  • CC License information is included at the bottom of the page (ok if user needs to scroll to view)
  • "Publish" publishes the alt text with an automatic edit summary "Added alt text"
  • Relevant events are instrumented according to Data instrumentation plan and completed slides have been marked with a green background
  • Translations are added for target languages from UI Copy for Suggested edits Alt text on iOS (T370725)
Design:

Figma:

image.png (375×667 px, 185 KB)

Engineering notes:
  • This should be one of the first priorities after MTV
  • Wikitext publishing should now happen at this step, not the half sheet as in the MTV

Event Timeline

Tsevener renamed this task from Add Review step for Alt text experiment and update publish to [L] Add Review step for Alt text experiment and update publish.Jul 23 2024, 6:16 PM
HNordeenWMF raised the priority of this task from Low to Medium.Jul 23 2024, 10:03 PM
Seddon set the point value for this task to 5.Jul 25 2024, 2:28 PM

Hi @scblr here we also have that accessibility icon that we cannot use in this context per Apple's guidelines. Should we remove it from here as well?

Can be tested on Wikipedia Experimental 7.5.8 (146)

Thanks @Mazevedo for the work on this 👏 I experienced this when trying it out:

Seems like there’s something off with the image crop/fit.

Seddon raised the priority of this task from Medium to High.Aug 19 2024, 8:33 PM

@scblr New design review build will be in Experimental 7.5.8 (149).

The only difference is the contents/length of the licensing information. Not quite sure what’s been decided here @Tsevener / @HNordeenWMF?

ImplementationvsDesign
IMG_7662.PNG (1,179×2,556 px, 1 MB)
image.png (375×667 px, 185 KB)

And likely two double spaces in that footer:

IMG_7662 copy.PNG (1,179×2,556 px, 1 MB)

Other than that, it looks good to me. No blockers from my end, I’ll move it forward.

@scblr @HNordeenWMF

The only difference is the contents/length of the licensing information.

Tagging @Mazevedo to confirm but it looks like we reused the footer from our editor edit summary screen:

Screenshot 2024-08-20 at 8.50.20 AM.jpeg (1,125×2,436 px, 541 KB)

Hi @scblr! I'm following the copy from the translation sheet. If I used the one in the designs, it wouldn't match the translations. For the extra space, I can remove it in the next task.

I'm following the copy from the translation sheet. If I used the one in the designs, it wouldn't match the translations. For the extra space, I can remove it in the next task.

Ok, that makes sense – thanks @Mazevedo @Tsevener

@Mazevedo this one is not appearing compatible with dynamic type. I'll make a task for adding it in the permanent version