Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mcoktwone-azuatcuvpc.now.sh - The ordered list numbering is faulty #29737

Closed
webcompat-bot opened this issue Apr 20, 2019 · 5 comments
Closed
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine priority-normal severity-minor The site has a cosmetic issue.
Milestone

Comments

@webcompat-bot
Copy link

URL: https://mcoktwone-azuatcuvpc.now.sh/wiki/teleporter

Browser / Version: Firefox 68.0
Operating System: Windows 10
Tested Another Browser: Yes

Problem type: Something else
Description: ol list not correct in nightly
Steps to Reproduce:
the list results correctly in 1,2,3,4,5 in Firefox stable 66,but it results in 1,4,5,6,7 in Firefox Nightly 68.
Screenshot Description

Browser Configuration
  • None

Reported by @jimchen5209

From webcompat.com with ❤️

@jimchen5209
Copy link

jimchen5209 commented Apr 20, 2019

Snipaste_2019-04-20_18-04-41
Browser used:

  • Top-left: Edge Chromium
  • Top-right: Firefox Nightly
  • Bottom-left: Edge UWP
  • Bottom-right: Chrome

@cipriansv cipriansv added priority-normal severity-minor The site has a cosmetic issue. labels Apr 30, 2019
@cipriansv cipriansv modified the milestones: needstriage, needsdiagnosis Apr 30, 2019
@cipriansv cipriansv changed the title mcoktwone-azuatcuvpc.now.sh - see bug description mcoktwone-azuatcuvpc.now.sh - The ordered list numbering is faulty Apr 30, 2019
@cipriansv
Copy link

Thanks for the report,
I was indeed able to reproduce the issue. The ordered list numbering is faulty on Firefox.

Tested with:
Browser / Version: Firefox Nightly 68.0a1 (2018-04-29), Chrome 74.0.3729.108
Operating System: Windows 10 Pro

This is the web page displayed in Firefox Nightly:

image

And this is the web page displayed in Chrome:

image

@miketaylr miketaylr added the engine-gecko The browser uses the Gecko rendering engine label Apr 30, 2019
@karlcow
Copy link
Member

karlcow commented May 3, 2019

Quite interesting. The markup is wrong.

<ol>
    <li></li>
    <ul></ul>
    <li><ul>
        <li></li></ul>
    </li>
    <li></li></ol>

The first ul should be inside li.

So it seems the way Firefox parses that is breaking the numbering. This is a test case to illustrate it.

data:text/html,<ol><li><a href="">item 1</a></li><ul><li><a href="">subitem 1.1</a></li><li><a href="">subitem 1.2</a></li><li><a href="">subitem 1.3</a></li></ul><li><a href="">item 2</a><ul><li><a href="">subtitem 2.1</a></li></ul></li><li><a href="">item 3</a></li></ol>

and indeed this is working in Firefox 66. Let's find a mozregression.

mozregression --good 66 --bad 2019-04-20

ah this one is for @emilio and @MatsPalmgren
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=5dc0652cd024fef30217ac3e6087b7a32f16967f&tochange=9bad25ccc8d0414e21d3c8d4dcf2178a97bb07c3

Opening a core bug.

@karlcow
Copy link
Member

karlcow commented May 3, 2019

Close as duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=1548753

@lock
Copy link

lock bot commented May 10, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue at https://webcompat.com/issues/new if you are experiencing a similar problem.

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine priority-normal severity-minor The site has a cosmetic issue.
Projects
None yet
Development

No branches or pull requests

5 participants