Opened 9 years ago
Last modified 3 years ago
#11009 assigned enhancement
[patch][cla][needs AMD conversion] coyotepoint-patch:chartImage-Require gif image support in charting
| Reported by: | sachin | Owned by: | dylan |
|---|---|---|---|
| Priority: | high | Milestone: | 1.15 |
| Component: | Charting | Version: | 1.4.2 |
| Keywords: | chart with custome image support | Cc: | cjolif |
| Blocked By: | Blocking: |
Description
Issue-In charting I was unable to show gif image instead of SVG.
Resolution- I created Custom.js with help of Default.js.I am assuming i'll get addSeries data in following format.
{x:6,y:4,image:'./images/alert.red.gif',width:20,height:20,tooltip:'tooltip message'}
I am attaching patch file as well as original code with testing html file.
Attachments (2)
Change History (18)
Changed 9 years ago by
| Attachment: | patch.chartImage added |
|---|
Changed 9 years ago by
| Attachment: | imageChart.zip added |
|---|
orignal files on 14.1 version with test html and out screen shot
comment:2 Changed 9 years ago by
What status should I switched to for this ticket? Do you want to retract it?
comment:3 Changed 9 years ago by
No I don't want retract it. I am expecting patch to be reviewed and if there are any comments,then we can discuss and I will change patch accordingly.
comment:4 follow-up: 5 Changed 9 years ago by
| Summary: | coyotepoint-patch:chartImage-Require gif image support in charting → [patch][cla ?] coyotepoint-patch:chartImage-Require gif image support in charting |
|---|
Need to verify CLA status... sachin, is this under your CLA or one for Coyote Point?
comment:5 Changed 9 years ago by
Replying to dylan:
Need to verify CLA status... sachin, is this under your CLA or one for Coyote Point?
For Coyote Point
comment:7 Changed 9 years ago by
| Summary: | [patch][cla ?] coyotepoint-patch:chartImage-Require gif image support in charting → [patch][cla] coyotepoint-patch:chartImage-Require gif image support in charting |
|---|
Confirmed, cla on file for Coyote Point.
comment:8 Changed 9 years ago by
| Milestone: | tbd → 1.6 |
|---|---|
| Status: | new → assigned |
comment:9 Changed 9 years ago by
| Milestone: | 1.6 → 1.7 |
|---|
The patch needs to be reworked because the codebase has changed. Other than that it looks good and we need this functionality.
comment:10 Changed 9 years ago by
| Cc: | eugene.lazutkin@… removed |
|---|
comment:11 Changed 8 years ago by
| Milestone: | 1.7 → 1.8 |
|---|
comment:12 Changed 7 years ago by
sachin, would you be able to update this to AMD / current charts for 1.8? Otherwise I fear it might be delayed one more time. Thanks.
comment:13 Changed 7 years ago by
| Cc: | cjolif added |
|---|
comment:14 Changed 7 years ago by
| Milestone: | 1.8 → 2.0 |
|---|
1.8 is frozen. Move all enhancements to next release. If you need an exemption from the freeze for this ticket, contact me immediately.
comment:15 Changed 4 years ago by
| Milestone: | 2.0 → 1.12 |
|---|---|
| Owner: | changed from Eugene Lazutkin to dylan |
| Summary: | [patch][cla] coyotepoint-patch:chartImage-Require gif image support in charting → [patch][cla][needs AMD conversion] coyotepoint-patch:chartImage-Require gif image support in charting |
We would consider this for 1.12 with a conversion to AMD.
comment:16 Changed 3 years ago by
| Milestone: | 1.13 → 1.15 |
|---|
Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.
Patch file