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

3D not rendering with styled 5.0.4 #6018

Closed
garybrain opened this Issue Nov 23, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@garybrain

garybrain commented Nov 23, 2016

Expected behaviour

Chart to be rendered in 3D

Actual behaviour

Chart showing in 2D due to JS error.

highcharts-3d.js:26 Uncaught TypeError: Cannot read property 'style' of undefined(…)(anonymous function)
@ highcharts-3d.js:26(anonymous function)
@ highcharts-3d.js:28(anonymous function)
@ highcharts-3d.js:8(anonymous function)
@ highcharts-3d.js:8

Live demo with steps to reproduce

Styled. ( error ).
using : https://code.highcharts.com/js/highcharts-3d.js
http://jsfiddle.net/cyn0xkk8/

Using following 3d.js works.
using : https://code.highcharts.com/highcharts-3d.js
http://jsfiddle.net/cyn0xkk8/1/

Affected browser(s)

All browsers

@pawelfus pawelfus added the Regression label Nov 23, 2016

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Nov 23, 2016

Contributor

Thank you for reporting!

This is a regression from 62003e6

Workaround:
Set defs.style.textContent before loading highcharts-3d.js file and add side/top CSS, Demo: http://jsfiddle.net/cyn0xkk8/2/

Snippet:

<script src="https://code.highcharts.com/js/highcharts.src.js"></script>
<script>
  Highcharts.setOptions({
    defs: {
      style: {
        textContent: ''
      }
    }
  });
</script>
<script src="https://code.highcharts.com/js/highcharts-3d.src.js"></script>

CSS:

.highcharts-3d-top {
  filter: url(#highcharts-brighter)
}

.highcharts-3d-side {
  filter: url(#highcharts-darker)
}
Contributor

pawelfus commented Nov 23, 2016

Thank you for reporting!

This is a regression from 62003e6

Workaround:
Set defs.style.textContent before loading highcharts-3d.js file and add side/top CSS, Demo: http://jsfiddle.net/cyn0xkk8/2/

Snippet:

<script src="https://code.highcharts.com/js/highcharts.src.js"></script>
<script>
  Highcharts.setOptions({
    defs: {
      style: {
        textContent: ''
      }
    }
  });
</script>
<script src="https://code.highcharts.com/js/highcharts-3d.src.js"></script>

CSS:

.highcharts-3d-top {
  filter: url(#highcharts-brighter)
}

.highcharts-3d-side {
  filter: url(#highcharts-darker)
}
@garybrain

This comment has been minimized.

Show comment
Hide comment
@garybrain

garybrain Nov 23, 2016

Many thanks

G,

garybrain commented Nov 23, 2016

Many thanks

G,

@Dams591

This comment has been minimized.

Show comment
Hide comment
@Dams591

Dams591 Nov 23, 2016

Hey guys
Was wondering if this will be fixed/pushed out with 5.0.5 and if so if you have a release date.
Cheers.

Dams591 commented Nov 23, 2016

Hey guys
Was wondering if this will be fixed/pushed out with 5.0.5 and if so if you have a release date.
Cheers.

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