Opened 7 years ago

Closed 4 years ago

#16141 closed enhancement (patchwelcome)

vertical axis seems to use wrong major tick sometimes

Reported by: ben hockey Owned by: Eugene Lazutkin
Priority: undecided Milestone: 1.13
Component: Charting Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description

it looks like if the smallest data point falls right on a major tick then the vertical axis cuts off at that point. this makes it look like there is no data for that point.

if the smallest data point falls on a major tick, should it use the next major tick below that?

i've attached a test case that shows the issue - my opinion is that the first column should look like it has data. if you change the smallest data point to something other than a multiple of 10 everything looks good.

Attachments (1)

16141.html (1.4 KB) - added by ben hockey 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by ben hockey

Attachment: 16141.html added

comment:1 Changed 7 years ago by cjolif

I guess this behavior makes a lot more sense when using a line plot for example. That makes y-axis starts at the lower value of the line plot which is reasonable. That said I fully agree it gives a quite non-intuitive result with a column plot as you have shown...

To me a column plot should never be used without includeZero: true on the y-axis. Indeed on the contrary of a line plot the column plot is really meant to be drawn from 0 (or at least from a user-defined baseline) to the data point value. Not from an abritrary automatically computed baseline. So I don't think I would personally face that issue.

Finally changing this in 1.x, in particular if that is done for all plots (which would be the "normal" way of doing it), would be problematic because this would cause a behavior change. (I'm thinking of the line plots where that behavior sounds quite logical).

For all of this, it seems to me more of the "enhancement" type that "defect" type.

comment:2 Changed 7 years ago by cjolif

Type: defectenhancement

comment:3 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: newclosed

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

Note: See TracTickets for help on using tickets.