Skip to content

/ highcharts

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

Jump to bottom

ignoreHiddenSeries causes column wrong position and width after legendItemClick #4921

Closed
KacperMadej opened this issue Jan 13, 2016 · 1 comment
Closed

ignoreHiddenSeries causes column wrong position and width after legendItemClick #4921

KacperMadej opened this issue Jan 13, 2016 · 1 comment
Assignees
@oysteinmoseng
Labels
Fix suggested Type: Bug
Milestone
5.0.12

Comments

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Jan 13, 2016

Example: http://jsfiddle.net/uct6eqga/
Steps:

  1. hide both series via legend
  2. show the black series via legend
  3. show the blue series via legend
  4. the black series is overlapping the blue one - is not adjusted

Might be related to #943.
@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Jan 15, 2016

Debugging hints: Look in the ColumnSeries getMetrics function. The columns are not supposed to go full width when the ignoreHiddenSeries is false, it is supposed to reserve space for any hidden series.
oysteinmoseng added a commit that referenced this issue Jan 20, 2016
@oysteinmoseng
Fixed #4921 ignoreHiddenSeries column width issue after legend click.
839aa26
oysteinmoseng added a commit that referenced this issue Apr 12, 2016
@oysteinmoseng
Added unit test for issue #4921.
7f18706
@oysteinmoseng oysteinmoseng added this to the 5.0.11 milestone Mar 29, 2017
@jon-a-nygaard jon-a-nygaard modified the milestones: 5.0.12, 5.0.11 May 18, 2017
TorsteinHonsi added a commit that referenced this issue May 23, 2017
@oysteinmoseng @TorsteinHonsi
Added unit test for issue #4921.
e75d04c
@pawelfus pawelfus mentioned this issue Feb 21, 2018
Closed
sigrunnu added a commit that referenced this issue Jun 26, 2019
@sigrunnu
Tests: Moved issue test #4921 to unit-tests.
4240ae8
@sigrunnu sigrunnu mentioned this issue Jun 26, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oysteinmoseng oysteinmoseng

Labels
Fix suggested Type: Bug
Projects
None yet
Milestone
5.0.12
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@TorsteinHonsi @jon-a-nygaard @oysteinmoseng @KacperMadej