Opened 11 years ago

Closed 11 years ago

#7522 closed enhancement (fixed)

setStroke with array argument for color fails on IE6+

Reported by: ptwobrussell Owned by: Eugene Lazutkin
Priority: high Milestone: 1.2
Component: DojoX GFX Version: 1.1.1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Eugene Lazutkin)

I noticed that this fails on IE6/7 for some reason:

ball = surface.createCircle({
    /* ... stuff ... */
})
//.setStroke("#888")
.setStroke([128,128,128])
.setFill("white")
;

It works fine on FF and Safari, but on IE it fails for both VML and Silverlight. Setting stroke with a string argument works fine though.

Change History (5)

comment:1 Changed 11 years ago by Eugene Lazutkin

Milestone: tbdfuture

Moving all open ticketd to the future.

comment:2 Changed 11 years ago by Eugene Lazutkin

Priority: normalhigh
Status: newassigned

comment:3 Changed 11 years ago by Eugene Lazutkin

Description: modified (diff)
Milestone: future1.2
Type: defectenhancement

Only string is supported as a shortcut, everything else (array, or object) should be properly wrapped:

shape.setStroke("red");
shape.setStroke({color: [255, 0, 0]});
shape.setStroke({color: {r: 255, g: 0, b: 0}});

I will extend this shortcut to include arrays, as well.

comment:4 Changed 11 years ago by Eugene Lazutkin

(In [15161]) gfx: allow to use an array as a shortcut for a stroke, the array will be treated as a color definition. Refs #7522. !strict

comment:5 Changed 11 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

(In [15162]) gfx: allow to use a dojo.Color as a shortcut for a stroke. Fixes #7522. !strict

Note: See TracTickets for help on using tickets.