Last Comment Bug 36717 - [LIST][INLINE-H]whitespace not ignored for list-style-position: inside
: [LIST][INLINE-H]whitespace not ignored for list-style-position: inside
Status: NEW
[CSS1-5.6.5]
: css1, testcase
Product: Core
Classification: Components
Component: Layout: Block and Inline (show other bugs)
: Trunk
: All All
: -- normal with 4 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2000-04-21 11:05 PDT by David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
Modified: 2014-12-15 09:57 PST (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
testcase described in bug (386 bytes, text/html)
2000-04-21 11:05 PDT, David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
no flags Details

Description David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2000-04-21 11:05:32 PDT
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
Comment 1 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2000-04-21 11:05:53 PDT
Created attachment 7817 [details]
testcase described in bug
Comment 2 troy 2000-04-21 11:13:45 PDT
Reassigning to buster
Comment 3 buster 2000-06-01 14:20:59 PDT
redistributing bugs across future milestones, sorry for the spam
Comment 4 buster 2000-06-12 18:10:35 PDT
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. 
Comment 5 Hixie (not reading bugmail) 2000-12-11 16:14:59 PST
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.
Comment 6 Kevin McCluskey (gone) 2001-10-04 16:32:24 PDT
Build reassigning Buster's bugs to Marc.
Comment 7 Greg K. 2002-07-11 00:37:12 PDT
Confirmed using FizzillaCFM/2002070913. Setting All/All.
Comment 8 Boris Zbarsky [:bz] 2003-04-20 11:22:56 PDT
.
Comment 9 Javi Rueda 2011-09-09 09:45:20 PDT
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 Javi Rueda 2011-09-25 19:16:46 PDT
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?

Note You need to log in before you can comment on or make changes to this bug.