1400211 - Articles that begin with a video are wrongly displayed in the "Recommended by Pocket" section

Status: RESOLVED WORKSFORME

(Firefox :: New Tab Page, defect)

Description

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: InnerHTML.txt

[Affected versions]:
- Firefox 57.0a1 Build ID 20170913220121

[Affected Platforms]:
- All Windows
- All Mac
- All Linux

[Prerequisites]:
- "browser.search.region" value is set to US. 
- Download the "InnerHTML" .txt document.

[Steps to reproduce]:
1. Start the browser with the profile from prerequisites and open a new tab.
2. Right click one of the "Recommended by Pocket" cards and click the Inspect element option.
3. Press right click on the node containing the website link and and paste the Inner HTML from the downloaded .txt document.
4. Observe how the card is displayed.

[Expected results]:
- The modified card is correctly displayed.

[Actual results]:
- The modified card does not have an image.

[Notes]:
- This issue is reproducible naturally, because the specific article appeared in the "Recommended by pocket" section.
- If the specific article (https://www.si.com/tech-media/2017/09/10/media-circus-nfl-opening-weekend-32-thoughts) appears in Highlights, an image is correctly displayed.

Comment 1

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: AS Recommended by pocket.gif

Screen recording of the issue.

Comment 2

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: As Recommended by Pocket SSS.png

Screen shot of this issue when is reproducible naturally.

Comment 3

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: As Recommended by Pocket SSS.png

Screen shot of this issue when is reproducible naturally.

Comment 4

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: New Tab with broken pocket card.zip

Full page saved as ".htm" where the issue is reproducible.

Comment 5

[Cristina Badescu [:cbadescu], Experiments QA]

Attachment: AS -screenshot.png

Attaching the correct screenshot.

Comment 6

[Ed Lee :Mardak]

Attachment: same article from pocket/explore/trending

(In reply to Cristina Badescu from comment #0)
> Created attachment 8908577 [details]
> InnerHTML.txt
If this was what came from pocket, it seems like there was no image associated to the article? `<div class="card-details no-image">`

The historical trending articles appear here https://getpocket.cdn.mozilla.net/explore/trending
And on that page, it looks like the css is broken: `style="background-image:  background-size:cover;"` which seems to indicate indeed there is no provided image.

Comment 7

[Matt Koidin]

Thanks for raising this issue.

(1) This was a mistake in our curation - the current guidance is not to include items without images.

(2) We've reviewed and understand why this happened. We are improving the curation tool to prevent items without images from getting posted to the feed going forward.

(3) Even though it shouldn't be displayed anymore - we've removed this specific item from the feed.

Let me know if I missed anything.

Comment 8

[Ed Lee :Mardak]

Indeed I don't see the article anymore on https://getpocket.cdn.mozilla.net/explore/trending

Comment 9

[Ed Lee :Mardak]

Attachment: article with no description

matt, not specifically this bug, but similar curation question. An article just showed up with no description. It looks okay, just different.

https://getpocket.cdn.mozilla.net/explore/trending?src=fx_new_tab#1891877363

The destination page has a description, so not sure why Pocket didn't provide one?

<meta property="og:description" content="Some members of the famously tech-shunning sect have begun to incorporate cellphones and computers into their lives, raising unique questions about the balance between work and home." />