Page MenuHomePhabricator

[S] Create modal sheet, next button and text field for Alt text MTV - flow B
Closed, ResolvedPublic

Assigned To
None
Authored By
HNordeenWMF
Jul 16 2024, 11:26 PM
Referenced Files
F57119504: CleanShot 2024-08-06 at 14.13.00@2x.png
Aug 6 2024, 12:24 PM
F57119201: image.png
Aug 6 2024, 12:11 PM
F57069429: image.png
Aug 5 2024, 12:55 PM
F57069427: image.png
Aug 5 2024, 12:55 PM
F57069425: image.png
Aug 5 2024, 12:55 PM
F57069422: image.png
Aug 5 2024, 12:54 PM
F56788514: Screenshot 2024-07-30 at 11.47.39.png
Jul 30 2024, 2:51 PM
F56470658: image.png
Jul 16 2024, 11:26 PM

Description

Background

When writing alt text, it's important that the user can review the image in the context of the article and its caption. To allow this, we would like to show the user the article view automatically scrolled to the location of the relevant image in the background, and a halfsheet with the input box for alt text in the foreground.

Requirements
  • Create initial version of sheet with title, text field for alt text, and next button
  • Halfsheet can be minimized and then expanded again
  • Article view behind halfsheet can be scrolled, image can be opened in gallery view from article view behind
  • "Describe the image" input box has a fixed height
  • This task represents the work to trigger the task from flow B only
Design:

Figma

image.png (667×375 px, 176 KB)
image.png (667×375 px, 186 KB)
image.png (667×375 px, 58 KB)
NOTE: Task is not intended to match designs fully, follow-up task T370327 will add image and link button, bottom copy/warning and counter, and guidance button

Event Timeline

Tsevener subscribed.

@HNordeenWMF I moved some requirements over to T370327, to help us get to MTV faster. Hope that's okay!

Tsevener renamed this task from Create sheet, next button and text field for Alt text MTV to [S] Create sheet, next button and text field for Alt text MTV.Jul 18 2024, 5:39 PM
Mazevedo renamed this task from [S] Create sheet, next button and text field for Alt text MTV to [S] Create modal sheet, next button and text field for Alt text MTV.Jul 23 2024, 3:15 PM
Mazevedo removed a subscriber: Jarrod_Baniqued.

Hi @scblr! Here on the modal header, the icon used is the SFSymbol named "accessibility." However, per Apple's guidelines, it "may only be used to refer to Apple's Accessibility features," so we can't use it here in this context.
Could we have an alternative to replace this icon?

Screenshot 2024-07-30 at 11.47.39.png (786×692 px, 69 KB)

Actually moving this to blocked, since we still need an icon.

HNordeenWMF raised the priority of this task from Low to Medium.Aug 1 2024, 9:33 PM

Hi @scblr! Here on the modal header, the icon used is the SFSymbol named "accessibility." However, per Apple's guidelines, it "may only be used to refer to Apple's Accessibility features," so we can't use it here in this context.
Could we have an alternative to replace this icon?

Screenshot 2024-07-30 at 11.47.39.png (786×692 px, 69 KB)

Great catch @Mazevedo; I suggest leaving out the icon completely:

It’s more calm, anyway.

FYI @HNordeenWMF – I also updated T370327.

Mazevedo renamed this task from [S] Create modal sheet, next button and text field for Alt text MTV to [S] Create modal sheet, next button and text field for Alt text MTV - flow B.Aug 5 2024, 3:53 PM
Mazevedo updated the task description. (Show Details)

Hey @Mazevedo 👋

I saw this note in the task’s description:

NOTE: Task is not intended to match designs fully, follow-up task T370327 will add image and link button, bottom copy/warning and counter, and guidance button

And it currently looks like this:

CleanShot 2024-08-06 at 14.13.00@2x.png (2×932 px, 565 KB)

So, I guess this is more or less how it’s intended.

However, I left a few notes regarding bugs/behavior in T370231#10044735 that also affect this one.