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

[Gecko Bug 1573907] Fix interaction of <li value> inside <ol reversed>. #18451

Merged
merged 2 commits into from Aug 15, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

The counting code was incrementing the start counter twice for

  • , as it
    generates both an increment and a reset.

    We should instead only increment it once per list-item, that is, once per
    increment.

    Differential Revision: https://phabricator.services.mozilla.com/D42001

    bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1573907
    gecko-commit: a9bdb526a7a01e0984c2bf8c888e8d8ffc69c5c4
    gecko-integration-branch: autoland
    gecko-reviewers: mats

  • The counting code was incrementing the start counter twice for <li value>, as it
    generates both an increment and a reset.
    
    We should instead only increment it once per list-item, that is, once per
    increment.
    
    Differential Revision: https://phabricator.services.mozilla.com/D42001
    
    bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1573907
    gecko-commit: a9bdb526a7a01e0984c2bf8c888e8d8ffc69c5c4
    gecko-integration-branch: autoland
    gecko-reviewers: mats
    Copy link
    Collaborator

    @wpt-pr-bot wpt-pr-bot left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Already reviewed downstream.

    Differential Revision: https://phabricator.services.mozilla.com/D42084
    
    bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1573907
    gecko-commit: 0bc9d6bfeb5a08b9dfe2f03c897e564478982e36
    gecko-integration-branch: autoland
    gecko-reviewers: mats
    @moz-wptsync-bot moz-wptsync-bot merged commit 1b240aa into master Aug 15, 2019
    @moz-wptsync-bot moz-wptsync-bot deleted the gecko/1573907 branch August 15, 2019 19:41
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    None yet

    3 participants