[LIST][INLINE-H]whitespace not ignored for list-style-position: inside

NEW
Unassigned

Status

()

Core
Layout: Block and Inline
17 years ago
2 years ago

People

(Reporter: dbaron, Unassigned)

Tracking

({css1, testcase})

Trunk
css1, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [CSS1-5.6.5])

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
DESCRIPTION:  The whitespace at the beginning of an <li> element should be
ignored, so that <li>stuff</li> and <li> stuff</li> look the same.  This is not
happening correctly for 'list-style-position: inside' (which places the list
marker inside the list item's box).

This is technically a parser issue, but it's rickg's oft-repeated position that
this should happen entirely inside layout.

STEPS TO REPRODUCE:
 * load attached testcase

ACTUAL RESULTS:
 * second list item separated from bullet more than first

EXPECTED RESULTS:
 * list items separated from bullet by the same amount

DOES NOT WORK CORRECTLY ON:
 * Linux, mozilla, 2000-04-20-09-M16
(Reporter)

Comment 1

17 years ago
Created attachment 7817 [details]
testcase described in bug

Comment 2

17 years ago
Reassigning to buster
Assignee: troy → buster

Updated

17 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M18

Comment 3

17 years ago
redistributing bugs across future milestones, sorry for the spam
Target Milestone: M18 → M19

Comment 4

17 years ago
This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another 
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration. 
Target Milestone: M19 → Future
(Reporter)

Updated

17 years ago
Summary: whitespace not ignored for list-style-position: inside → [LIST][INLINE-H]whitespace not ignored for list-style-position: inside
Keywords: css1
Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.
Keywords: testcase
Build reassigning Buster's bugs to Marc.
Assignee: buster → attinasi
Status: ASSIGNED → NEW
Whiteboard: [CSS1-5.6.5]

Comment 7

15 years ago
Confirmed using FizzillaCFM/2002070913. Setting All/All.
OS: Linux → All
Hardware: PC → All
.
Assignee: attinasi → block-and-inline
Component: Layout → Layout: Block & Inline
QA Contact: petersen → ian
Priority: P3 → --
Target Milestone: Future → ---
Assignee: layout.block-and-inline → nobody
QA Contact: ian → layout.block-and-inline

Updated

6 years ago
Assignee: nobody → leofigueres

Comment 9

6 years ago
Try to modify text, removing the leading space here: http://mxr.mozilla.org/mozilla-central/source/layout/generic/nsBulletFrame.cpp#384

Or change the position of the bullet?

Comment 10

6 years ago
Forget about comment 9. Bad file, and bug description states that white-space must be ignored. So I am trying with modifying the string. Any clue where to look at the source code?

Updated

6 years ago
Assignee: leofigueres → nobody
You need to log in before you can comment on or make changes to this bug.