Closed Bug 306504 Opened 19 years ago Closed 19 years ago

Numbers with decimal point but no digits following considered valid

Categories

(Core :: CSS Parsing and Computation, defect, P2)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9alpha1

People

(Reporter: bjoern, Assigned: dbaron)

Details

(Keywords: fixed1.8, Whiteboard: [patch])

Attachments

(2 files)

test case
238 bytes, text/html
Details
patch
3.78 KB, patch
bzbarsky
: review+
bzbarsky
: superreview+
cbeard
: approval1.8b4+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.2; SV1)
Build Identifier: 

Deer Park Alpha 2 fails to ignore declarations like

padding: 20.px 20.px 20.px 20.px

which are not allowed per CSS 2.1 4.3.1. IE6 and
Opera8 ignore such declarations properly.

Reproducible: Always

Steps to Reproduce:
Attached file test case — 
Summary: Numbers lacking fraction considered valid → Numbers with decimal point but no digits following considered valid

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch
        — 
          — Splinter Review
      

    


  
This also removes the unused and vaguely-dangerous-looking IS_ALPHA.

        Attachment #194377 -
        Flags: superreview?(bzbarsky)

        Attachment #194377 -
        Flags: review?(bzbarsky)
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Windows Server 2003 → All
Priority: -- → P2
Hardware: PC → All
Whiteboard: [patch]
Target Milestone: --- → mozilla1.9alpha

        Attachment #194377 -
        Flags: superreview?(bzbarsky)

        Attachment #194377 -
        Flags: superreview+

        Attachment #194377 -
        Flags: review?(bzbarsky)

        Attachment #194377 -
        Flags: review+

    
    

    
  
Fix checked in to trunk, 2005-08-31 9:15 -0700.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

    
    

    
  
Comment on attachment 194377 [details] [diff] [review]
patch

Very low risk standards-compliance fix.

        Attachment #194377 -
        Flags: approval1.8b4?

    
  

        Attachment #194377 -
        Flags: approval1.8b4? → approval1.8b4+

    
  
Flags: blocking1.8b5+
Flags: blocking1.8b4+

    
    

    
  
Fix checked in to MOZILLA_1_8_BRANCH, 2005-08-31 14:23 -0700.
Keywords: fixed1.8

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: