search

kmputin / dwpe

Automatically exported from code.google.com/p/dwpe
MIT License
0 stars 0 forks source link

Better support for container #39

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
If specifying a container, the charts are not style properly. I would suggest 
changing this code:

//create canvas wrapper div, set inline w&h, append
var canvasContain = (container || $('<div class="visualize" role="img" 
aria-label="Chart representing data from the table: '+ title +'" />'))
            .height(o.height)
            .width(o.width)
            .append(canvas);

To 

//create canvas wrapper div, set inline w&h, append
var canvasContain = (container || $('<div/>'));
canvasContain.addClass('visualize').attr('role', 'img').attr('aria-label', 
'Chart representing data from the table: ' + title)

           canvasContain.height(o.height)
            .width(o.width)
            .append(canvas);

Original issue reported on code.google.com by Laurent....@gmail.com on 16 Sep 2010 at 3:09