990318 - list-style-position: inside should not indent texts if list-style-type is none

Status: NEW

(Core :: Layout: Block and Inline, defect)

Description

[Takeshi Kurosawa]

Attachment: Testcase

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release)
Build ID: 20140331030201

Steps to reproduce:

Open an HTML file which contains elements with "list-style-position: inside" and "list-style-type: none".


Actual results:

<li> elements with "list-style-position: inside" indent their texts even if "list-style-type" is "none" in Firefox 28 and Firefox Nightly 31.0a1 (2014-03-31).

<li> elements without "list-style-position" and with "list-style-type: none" don't indent texts.


Expected results:

In Firefox 27, Internet Explorer 10 and Google Chrome 33, both kinds of <li> elements don't indent texts and their behaviors seem to be natural.

Related bugs: Bug 645647?

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Probably a regression from bug 934072, although I haven't verified this.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Also, I gave you canconfirm and editbugs Bugzilla privileges; see https://developer.mozilla.org/en-US/docs/What_to_do_and_what_not_to_do_in_Bugzilla

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

(In reply to David Baron [:dbaron] (needinfo? me) (UTC-7) from comment #1)
> Probably a regression from bug 934072, although I haven't verified this.

I agree this is a regression from bug 934072. However, I think the patch mostly handles the padding correctly. The only thing we have to do, IMO, is suppressing padding for none in addition.

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

I have a question that, what should we do if the author explicitly set padding or margin for ::marker (currently ::-moz-list-{bullet,number}) when 'list-style-type' is 'none'?

According to the css3-lists spec, I guess we should never generate box for 'none'-styled list, which means no padding or margin should be applied on it.

Comment 5

[Xidorn Quan [:xidorn] UTC+10]

This patch I submitted for bug 1013160 also appears to fix this bug as it removes the default padding in -moz-list-{bullet,number}. But comment 4 is still a question.

Comment 6

[Takeshi Kurosawa]

I confirmed the original issue was fixed by bug 1013160.