Opened 7 years ago

Closed 4 years ago

#16068 closed defect (patchwelcome)

dojox.charting: chart hides borders of parent div container/ charting area

Reported by: Paul Christopher Owned by: cjolif
Priority: low Milestone: 1.13
Component: Charting Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description

Description

Parts of the charting area's border are hidden by the chart:

Steps to reproduce the issue

Change dojox/charting/tests/test_rotatedLabels.html and simply add a border to <div id="test1"> like so:

<div id="test1" style="width: 600px; height: 400px; border: 5px solid #000;"></div>

Run the test case. This gives the results shown in the above screenshot (tested IE9, FF15).

Workaround

You need to wrap the charting area in another div and give that div a border.

Attachments (2)

chart.png (32.7 KB) - added by Paul Christopher 7 years ago.
test_rotatedLabels.html (3.2 KB) - added by Paul Christopher 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by Paul Christopher

Attachment: chart.png added

Changed 7 years ago by Paul Christopher

Attachment: test_rotatedLabels.html added

comment:1 Changed 7 years ago by cjolif

Owner: changed from Eugene Lazutkin to cjolif
Priority: undecidedlow
Status: newassigned

comment:2 Changed 7 years ago by cjolif

Milestone: tbd2.0

Yes I think Charting should use contentBox instead of marginBox as it does now to size itself. This would avoid this issue. As this would be slightly incompatible I would however think about waiting for 2.0 to do that. In particular as the workaround is an easy one.

comment:3 Changed 7 years ago by cjolif

See also #16567

comment:4 Changed 4 years ago by dylan

Milestone: 2.01.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 3+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.