Closed Bug 291262 Opened 19 years ago Closed 19 years ago

styling the CANVAS element doesn't work

Categories

(Core :: Graphics: Canvas2D, defect)

Product:
Core
Component:
Graphics: Canvas2D
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: annevk, Assigned: vlad)

Details

(Keywords: testcase)

Attachments

(3 files)

testcase - HTML
306 bytes, text/html
Details
testcase - XHTML
311 bytes, application/xml
Details
reftest
2.38 KB, patch
dbaron
: review+
dbaron
: superreview+
Details | Diff | Splinter Review 
Per WA 1.0 - or HTML5 - you should be able to style the CANVAS element.
Attached file testcase - HTML — 

    
    

    
  

      
      
      
      

        Attached file
          
        testcase - XHTML
        — 
      

    


  

    
    

    
  
This is not "per WA1" or "per HTML5", it's per the CSS spec. Replaced elements
are styleable.

    
    

    
  
This has nothing to do with view rendering, really...  Do we need a separate
component for canvas bugs?  They might get the right assignee too, then.
Assignee: roc → vladimir
Flags: blocking1.8b3?

    
    

    
  
You got it.
Component: Layout: View Rendering → Layout: Canvas

    
    

    
  
should be fixed by checkin for bug 260120
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

    
    

    
  
Verified.

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050507
Firefox/1.0+
Status: RESOLVED → VERIFIED

    
  
Flags: blocking1.8b3?

    
    

    
  

      
      
      
      

        Attached patch
          
          
        reftest
        — 
          — Splinter Review
      

    


  
testcase presented as reftest

        Attachment #259136 -
        Flags: review?(vladimir)

    
  

        Attachment #259136 -
        Flags: review?(vladimir) → review?(dbaron)

    
    

    
  
Comment on attachment 259136 [details] [diff] [review]
reftest

r+sr=dbaron, but why not use different positions than top:0 and left:0 (which seems like they could accidentally match something else)  How about top:37px; left:4em; ?

        Attachment #259136 -
        Flags: superreview+

        Attachment #259136 -
        Flags: review?(dbaron)

        Attachment #259136 -
        Flags: review+

    
  
Keywords: checkin-needed

    
    

    
  
RCS file: /cvsroot/mozilla/layout/reftests/bugs/291262-1-ref.html,v
done
Checking in layout/reftests/bugs/291262-1-ref.html;
/cvsroot/mozilla/layout/reftests/bugs/291262-1-ref.html,v  <--  291262-1-ref.html
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/layout/reftests/bugs/291262-1.html,v
done
Checking in layout/reftests/bugs/291262-1.html;
/cvsroot/mozilla/layout/reftests/bugs/291262-1.html,v  <--  291262-1.html
initial revision: 1.1
done
Checking in layout/reftests/bugs/reftest.list;
/cvsroot/mozilla/layout/reftests/bugs/reftest.list,v  <--  reftest.list
new revision: 1.327; previous revision: 1.326
done
Flags: in-testsuite+
Keywords: checkin-needed

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