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

Word cloud does not render in IE11 #8652

Closed
philclaridge opened this Issue Jul 19, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@philclaridge

philclaridge commented Jul 19, 2018

Expected behaviour

Render workclound

Actual behaviour

Graph title rendered but not wordcloud.

Live demo with steps to reproduce

Navigate to your highcharts demos https://www.highcharts.com/demo/wordcloud in IE11 - see screenshots - wordcloud not rendered

Validated error using IE11 using multiple sites including MS test image at:
Microsoft IE11 / Win 7 test image at https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/

ie11

Tested using free account for IE9 (but is IE11?): at https://www.browserling.com/

ie9

Product version

Highcharts current

Affected browser(s)

IE11. Other IE versions not tested.

@philclaridge philclaridge changed the title from Word cloud does not render in IE 9/IE11 to Word cloud does not render in IE11 Jul 19, 2018

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Jul 20, 2018

Contributor

Hi @philclaridge

Thank you for reporting about the problem.

The issue happens on any IE, because IE doesn't support Number.MIN_SAFE_INTEGER nor Number.MAX_SAFE_INTEGER used by getBoundingBoxFromPolygon @jon-a-nygaard

left: Number.MAX_SAFE_INTEGER,
right: Number.MIN_SAFE_INTEGER,
bottom: Number.MIN_SAFE_INTEGER,
top: Number.MAX_SAFE_INTEGER

Contributor

KacperMadej commented Jul 20, 2018

Hi @philclaridge

Thank you for reporting about the problem.

The issue happens on any IE, because IE doesn't support Number.MIN_SAFE_INTEGER nor Number.MAX_SAFE_INTEGER used by getBoundingBoxFromPolygon @jon-a-nygaard

left: Number.MAX_SAFE_INTEGER,
right: Number.MIN_SAFE_INTEGER,
bottom: Number.MIN_SAFE_INTEGER,
top: Number.MAX_SAFE_INTEGER

@KacperMadej KacperMadej added the Bug label Jul 20, 2018

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Jul 20, 2018

Contributor

Workaround:
Add a polifill like:

// polifill
Number.MAX_SAFE_INTEGER = 9007199254740991;
Number.MIN_SAFE_INTEGER = -9007199254740991;

Demo: https://codepen.io/anon/pen/GBNwdo

Contributor

KacperMadej commented Jul 20, 2018

Workaround:
Add a polifill like:

// polifill
Number.MAX_SAFE_INTEGER = 9007199254740991;
Number.MIN_SAFE_INTEGER = -9007199254740991;

Demo: https://codepen.io/anon/pen/GBNwdo

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