As of today's build
FF Nightly version : 67.0a1 (2019-02-10)
OS : Mac and Windows 10 Pro

Latest end-point dev-test-all has incorrect numbers for all numbered list sections.

I would guess bug 1519879 regressed this with the changes to the html structure of lists?

To test:

1. Set the pref browser.newtabpage.activity-stream.discoverystream.config to {"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}
2. Go to about:home
3. Scroll down a bit until you see a numbered list item. (Should be just below the fold, three numbers items with circle blue list numbers)

Expected: Should see a list going 1, 2, 3.
Actual: See a list with items going 1, 1, 1.

This is caused by the spoc changes to List, we added the intersection observer component around the list item element, which now breaks it out of the css needed for content: counter(list); which seems to make it think it's a bunch of 1 item lists.

One initial option is to swap the li element and the intersection observer component's order, seems to fix it and list spocs still works?

Clearing NI, thanks Scott for picking this one!

[Tracking Requested - why for this release]:

Fixes numbering list regressed in bug 1519879 fix.

https://github.com/mozilla/activity-stream/pull/4782

https://hg.mozilla.org/mozilla-central/rev/6d9b468951ce

Brahmini, can you verify that the list numbers are fixed on nightly? See steps here: https://bugzilla.mozilla.org/show_bug.cgi?id=1526861#c3

Can you add an uplift request for this as well?

Comment on attachment 9043739 [details]
Bug 1526861 - Incorrect numbers listed for numbered list sections

Beta/Release Uplift Approval Request

Feature/Bug causing the regression

Bug 1519879

User impact if declined

User will see list items incorrectly numbered

Is this code covered by automated tests?

No

Has the fix been verified in Nightly?

Yes

Needs manual test from QE?

Yes

If yes, steps to reproduce

1. Set the pref browser.newtabpage.activity-stream.discoverystream.config to {"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}
2. Go to about:home
3. Scroll down a bit until you see a numbered list item. (Should be just below the fold, three numbers items with circle blue list numbers)

Expected: Should see a list going 1, 2, 3.
Actual: See a list with items going 1, 1, 1.

List of other uplifts needed

Bug 1519879

Risk to taking this patch

Low

Why is the change risky/not risky? (and alternatives if risky)

Very low risk small patch. Fixes regression from 1519879 patch that displays sponsored content in list for discovery new tab experiments.

String changes made/needed

None

I have verified in Nightly 67.0a1 on Mac OS 10.14.3 that the numbers for the numbered list module are being displayed correctly!

Comment on attachment 9043739 [details]
Bug 1526861 - Incorrect numbers listed for numbered list sections

Planned work for pocket/new tab. Verified in Nightly.
OK for beta uplift, should land for beta 9.
Landing order: bug 1519879, bug 1525494, bug 1526861, bug 1524669, bug 1527195, bug 1525391, bug 1527347, bug 1525366, bug 1527626, bug 1527397, bug 1518258, bug 1527701, bug 1527370.

https://hg.mozilla.org/releases/mozilla-beta/rev/5d44e50ff0c2

(In reply to Wolasi from comment #14)

I have verified in Nightly 67.0a1 on Mac OS 10.14.3 that the numbers for the
numbered list module are being displayed correctly!

Can you please verify this issue on Firefox 66 Beta 9 (https://archive.mozilla.org/pub/firefox/candidates/66.0b9-candidates/build1/)?

I have verified in Beta 66.0b9 on Mac OS 10.14.3 that the numbers for the numbered list module are being displayed correctly

(In reply to Wolasi from comment #18)

I have verified in Beta 66.0b9 on Mac OS 10.14.3 that the numbers for the
numbered list module are being displayed correctly

Thanks for verifying this!