Page MenuHomePhabricator

Add image - suggested audio files (ogg) break insert image UI
Closed, DuplicatePublic

Description

On testwiki wmf.9 - this article has the ogg file as a suggested image.

The reason was stated as "One of the several images linked to the Wikidata item for this article" - and the file on wikidata - https://www.wikidata.org/wiki/File:Avoidant_personality_disorder.ogg - is linked only to the following page: https://en.wikipedia.org/wiki/File:Avoidant_personality_disorder.ogg.

When the file is suggested as add image, clicking 'yes' will display a broken insert image/caption layout:

Screen Shot 2021-11-18 at 5.37.25 PM.png (1×752 px, 211 KB)
Screen Shot 2021-11-18 at 5.37.34 PM.png (1×742 px, 71 KB)

(click on the animated gif below)

audio_files_insert.gif (669×408 px, 337 KB)

Event Timeline

Should non-image file types be excluded from the recommendation API in the first place?

Should non-image file types be excluded from the recommendation API in the first place?

That was my assumption too.

@mewoph @Etonkovidova -- we encountered this very same issue with video files. @Tgr also recommended that we just keep these pairings (article+video or article+audio) out of the queue. It's something that really should happen upstream with the algorithm, and so I'm going to save this as a requirement for the next iteration of the API. In the meantime, I think we should ignore this bug for Iteration 1 since it will be rare.

We can show a "no suggestions" error, that would be pretty easy. It would stay in the queue though, which is not a great experience. With the current behavior, you can accept or reject it, and while the editing interface won't look particularly good (I think the weird narrow shape at least would be fixed by the patch in T295922: [testwiki-wmf.9] Add image - small size image breaks caption box), it could be accepted/rejected, and the edit would work just fine.

@mewoph @Etonkovidova -- we encountered this very same issue with video files. @Tgr also recommended that we just keep these pairings (article+video or article+audio) out of the queue. It's something that really should happen upstream with the algorithm, and so I'm going to save this as a requirement for the next iteration of the API. In the meantime, I think we should ignore this bug for Iteration 1 since it will be rare.

We can show a "no suggestions" error, that would be pretty easy. It would stay in the queue though, which is not a great experience. With the current behavior, you can accept or reject it, and while the editing interface won't look particularly good (I think the weird narrow shape at least would be fixed by the patch in T295922: [testwiki-wmf.9] Add image - small size image breaks caption box), it could be accepted/rejected, and the edit would work just fine.

Is it possible to filter out audio files but keep video? I can see videos potentially being expected and a match, but audio files for an "Add a suggested image" task is quite unexpected.

We can treat audio and video differently if needed.

We can treat audio and video differently if needed.

Great, thanks. Per our team discussion earlier I've created a next iteration task T296220 for handling video.

Can we close this task given T296333: Add an image: exclude non-images is being done for Iteration 1?