1470150 - noscript.css should only apply to HTML elements.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P3)

Description

[Emilio Cobos Álvarez [:emilio]]

noscript.css applies to non-HTML elements since bug 1169514.

Comment 1

[Phabricator Automation]

Attachment: Bug 1470150: noscript rules should not apply to non-HTML elements. r=heycam

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Comment on attachment 8986767 [details]
Bug 1470150: noscript rules should not apply to non-HTML elements. r=heycam

(Just in case you still don't have Phabricator stuff completely set up :))

Comment 3

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1470150: noscript rules should not apply to non-HTML elements.

Test Plan: wpt

Reviewers: heycam

Tags: #secure-revision

Bug #: 1470150

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

Review commit: https://reviewboard.mozilla.org/r/252078/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/252078/

Comment 4

[Phabricator Automation]

Comment on attachment 8986767 [details]
Bug 1470150: noscript rules should not apply to non-HTML elements. r=heycam

Cameron McCormack (:heycam) has approved the revision.

https://phabricator.services.mozilla.com/D1749

Comment 5

[Cameron McCormack (:heycam)]

Comment on attachment 8986831 [details]
Bug 1470150: noscript rules should not apply to non-HTML elements.

Thanks, looks like my phabricator is working OK now.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e27f16329c72
noscript rules should not apply to non-HTML elements. r=heycam

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/11623 for changes under testing/web-platform/tests

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/e27f16329c72

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged