JeremyJStarcher / highlanddots

Automatically exported from code.google.com/p/highlanddots
0 stars 0 forks source link

Volta bounding boxes incorrect #16

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The bounding box for voltas is calculated improperly, causing the software
to crash during painting of boundary boxes.

Original issue reported on code.google.com by Jeremy.Starcher@gmail.com on 3 May 2010 at 5:12

GoogleCodeExporter commented 9 years ago
What browser and OS? On my FF3.5/XPsp3 combo the painting doesn't crash, but the
voltas aren't painted either.
Clearly there's a bug, I just want to determine if it's a simple painting issue 
or a
potentially nasty x-browser problem.

Original comment by pipemakertjm@gmail.com on 4 May 2010 at 2:14

GoogleCodeExporter commented 9 years ago
Should we change issue 12 to 'defect' and deal with issues 12 and 16 at the 
same time?

Original comment by pipemakertjm@gmail.com on 4 May 2010 at 2:16

GoogleCodeExporter commented 9 years ago
Right now I'm on Linux, Fx3.5.9, but I get the same issue with Sea Monkey.
While painting the bounding boxes, it crashes it hdots.js, here:

            mel.rect = rect;
                           if (doPaint && drawBoundingBox) {
                             ctx.fillStyle = "rgba(0, 0, 200, 0.5)";
                             ctx.fillRect(rect.x, rect.y, rect.width, rect.height); 
// Crash and burn
                             ctx.fillStyle = strokeStyle;

So rect isn't being returned correctly from voltas.

I don't think that issue 12 and 16 are really related, so I'd keep them 
separate.

Original comment by Jeremy.Starcher@gmail.com on 4 May 2010 at 4:18