New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PlotBand border on on three out of four sides of rect #5909

Closed
dbasner opened this Issue Oct 31, 2016 · 4 comments

Comments

Projects
None yet
5 participants
@dbasner

dbasner commented Oct 31, 2016

Unsure if this is purposeful or not, but having plotBorder give a three sided border is a little confusing. It kind of gives the impression that the rectangle runs longer than the length of the chart and just isn't being rendered properly. I feel like the border should either run on all four sides, or alternatively, run on the sides related to the axis that the plot band runs on (x axis plot bands on "left" and "right" side, y axis plot bands on the "top" and "bottom" of the rectangle).

Expected behaviour

A plotBand on either the x or y axis would draw a rectangle, and adding a border would show a visible border on all four sides (or two of them to indicate the bounds of that area).

Actual behaviour

Border only appears on three sides

Live demo with steps to reproduce

http://jsfiddle.net/dbasner/vpkv81au/1/

Affected browser(s)

@oysteinmoseng

This comment has been minimized.

Show comment
Hide comment
@oysteinmoseng

oysteinmoseng Nov 1, 2016

Collaborator

Thank you for your feedback. Internally all four sides currently have a border, but because of z-indexing only three sides have visible borders.

Collaborator

oysteinmoseng commented Nov 1, 2016

Thank you for your feedback. Internally all four sides currently have a border, but because of z-indexing only three sides have visible borders.

@Izothep

This comment has been minimized.

Show comment
Hide comment
@Izothep

Izothep Sep 28, 2017

Bug looks to be reintroduced in 5.0.10. Highcharts version.

It is probably related to below commit: 2b98675

Izothep commented Sep 28, 2017

Bug looks to be reintroduced in 5.0.10. Highcharts version.

It is probably related to below commit: 2b98675

@Izothep Izothep reopened this Sep 28, 2017

@torstenlabs

This comment has been minimized.

Show comment
Hide comment
@torstenlabs

torstenlabs Sep 28, 2017

Hey,
does anybody when it will be fixed?

Regards

Torsten

torstenlabs commented Sep 28, 2017

Hey,
does anybody when it will be fixed?

Regards

Torsten

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Oct 3, 2017

Contributor

Hi Torsten,

The problem is fixed and will be published with next version release.

Contributor

KacperMadej commented Oct 3, 2017

Hi Torsten,

The problem is fixed and will be published with next version release.

@KacperMadej KacperMadej removed the Undecided label Oct 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment