Opened 9 years ago

Closed 9 years ago

#12221 closed enhancement (fixed)

[patch][CCLA]Leverage Canvas Text in Charting

Reported by: cjolif Owned by: Eugene Lazutkin
Priority: high Milestone: 1.6
Component: Charting Version: 1.6.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

Once ​http://trac.dojotoolkit.org/ticket/12158 will be committed, canvas will supports text.

That means that specific fallback in charting to use html labels on canvas in all cases can be removed.

Also when rotation was asked with html labels, CSS3 transform was used to rotate the html label. However as the only case where html labels were used _with_ rotation was on canvas. Now that canvas use GFX labels that CSS3 transform code is not needed anymore.

Attachments (1)

Default.patch​ (2.1 KB) - added by cjolif 9 years ago.

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by cjolif

Attachment: Default.patch​ added

comment:1 Changed 9 years ago by cjolif

[patch][ccla] (forgot it in the title)

comment:2 Changed 9 years ago by Chris Mitchell

Summary: Leverage Canvas Text in Charting → [patch][CCLA]Leverage Canvas Text in Charting

comment:3 Changed 9 years ago by Eugene Lazutkin

Milestone: tbd → 1.6
Status: new → assigned

comment:4 Changed 9 years ago by Eugene Lazutkin

Resolution: → fixed
Status: assigned → closed

(In [23688]) charting: now canvas supports text => remove unneeded code, thx Christophe Jolif (IBM, CCLA), !strict, fixes #12221.

Note: See TracTickets for help on using tickets.