Closed Bug 272547 Opened 20 years ago Closed 19 years ago

ol li overflow:auto sets counter to zero

Categories

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

Product:
Core
Component:
Layout: Block and Inline
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: felix-sein-hosenladen, Assigned: roc)

References

Details

(Keywords: css2, testcase)

Attachments

(2 files)

testcase from #0 as attachment
(deleted), text/html
Details
fix
(deleted), patch
bzbarsky
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

The css-rule "overflow:auto" sets the li counter to 0. Every li now has the
number 0.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>abc</title>

<style type="text/css" media="screen">
li {
  width:100px;
  list-style-type:decimal;
  list-style-position:inside;
  overflow:auto;
}
</style>

</head>
<body>

<ol>
<li>aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa</li>
<li>bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb</li>
<li>cccccccccccccccccccccccccccccccccccccccccccccccccccccc</li>
</ol>

</body>
</html>

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
I see this behaviour here too, using the following builds:

Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; de-AT; rv:1.8a4) Gecko/20040927
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; de-DE; rv:1.7.5) Gecko/20041108
Firefox/1.0
Assignee: firefox → nobody
Component: General → Layout
OS: Windows 2000 → All
Product: Firefox → Core
QA Contact: firefox.general → core.layout
Hardware: PC → All
Version: unspecified → Trunk
Attached file testcase from #0 as attachment (deleted) — 

    
    

    
  
this might be the better component for this, correct me if I am wrong.
Assignee: nobody → dbaron
Component: Layout → Style System (CSS)
QA Contact: core.layout → ian

    
  
Status: UNCONFIRMED → NEW
Ever confirmed: true

    
  
Depends on: 4522

    
  
Keywords: css2, 
          
            testcase
Assignee: dbaron → nobody
Component: Style System (CSS) → Layout: Block and Inline
QA Contact: ian → layout.block-and-inline
Blocks: 282569

    
    

    
  

      
      
      
      

        Attached patch
          
          
        fix (deleted)
        — 
          — Splinter Review
      

    


  
The fix is fairly easy ... when looking for blocks to renumber into, we have to
skip through any wrappers (nsHTMLScrollFrame in this case).
Assignee: nobody → roc
Status: NEW → ASSIGNED

        Attachment #178427 -
        Flags: superreview?(bzbarsky)

        Attachment #178427 -
        Flags: review?(bzbarsky)

    
    

    
  
Comment on attachment 178427 [details] [diff] [review]
fix

r+sr=bzbarsky

        Attachment #178427 -
        Flags: superreview?(bzbarsky)

        Attachment #178427 -
        Flags: superreview+

        Attachment #178427 -
        Flags: review?(bzbarsky)

        Attachment #178427 -
        Flags: review+

    
    

    
  
checked in
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: