782551 - Drop dd text-indent quirk

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Simon Pieters [:zcorpan] (OOO until July 29)]

Bug 5119 should be reverted. The HTML spec deliberately omits this quirk. Opera and WebKit don't have it, and possibly IE10's non-compat-view quirks mode doesn't have it, either (I haven't checked). Opera does not plan on implementing this quirk.

[[
dd { margin-left: 40px; } /* LTR-specific: use 'margin-right' for rtl elements */
]]
http://www.whatwg.org/specs/web-apps/current-work/multipage/rendering.html#lists

Comment 1

[Boris Zbarsky [:bzbarsky]]

We should probably do this; this can cause performance problems, and if no one else has the quirk, we really shouldn't either, right?

Comment 2

[Emilio Cobos Álvarez (:emilio)]

So we're talking about these two rules right?

Sure, I agree that if nobody else does it, we shouldn't either... We should probably also revert bug 8749 while we're at it. Other browsers don't seem to have that quirk either.

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 782551 - Remove two quirks that other browsers don't have.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b1133e4b7e95
Remove two quirks that other browsers don't have. r=bzbarsky

Comment 5

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

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

Comment 6

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

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

[Natalia Csoregi [:nataliaCs]]

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