Skip to content
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

Drillup button with long text is not properly positioned in Highcharts v6 #7898

Closed
DanielRLevine opened this Issue Feb 21, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@DanielRLevine
Copy link

DanielRLevine commented Feb 21, 2018

Expected behaviour

When given the settings

drillUpButton: {
    relativeTo: 'spacingBox',
    position: {
        x: 0,
        y: 30
    }

drillup button should appear fully within the window.

Actual behaviour

In Highcharts v5, drillup button DOES appear fully within the window. In Highcharts v6, drillup button overflows out of the window when its text is long.

Live demo with steps to reproduce

http://jsfiddle.net/LLExL/9038/

To reproduce:

  1. Click on United States of America
  2. Click on any state
  3. The drillup button will overflow out of the window

Product version

Highcharts: v6.0.7
Maps Module: v6.0.7
Drilldown Module: v6.0.7

Affected browser(s)

Reproduced in Chrome, Firefox, and Safari.

Thank you!!!

@KacperMadej

This comment has been minimized.

Copy link
Contributor

KacperMadej commented Feb 22, 2018

Hi @DanielRLevine

Thank you for reporting about the problem.

As a workaround you could use a wrapper added in this demo: http://jsfiddle.net/BlackLabel/LLExL/9066/
The workaround is based on removing a fix for another problem that was resolved quite long ago, so it's hard to be sure if the workaround will break something important for you - pleas test it.

Related issue with the problem for which the fix is removed in the workaround: #7166

Internal note: offending commit c4fef24

@KacperMadej KacperMadej added the Bug label Feb 22, 2018

@TorsteinHonsi

This comment has been minimized.

Copy link
Collaborator

TorsteinHonsi commented Feb 28, 2018

Simplified demo: http://jsfiddle.net/v2xmgvrm/1/. Click the First and the Second columns successively.

Broken down to the SVGRenderer: http://jsfiddle.net/v2xmgvrm/12/

@TorsteinHonsi TorsteinHonsi added Regression and removed Bug labels Feb 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.