/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

IE11 + compatibility mode (IE 7 and 5): hover error #7800

Closed
KacperMadej opened this Issue Feb 5, 2018 · 6 comments

Comments

Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
3 participants
@KacperMadej @zlateskud @pawelfus
@KacperMadej
Contributor

KacperMadej commented Feb 5, 2018

Behaviour

When running Highcharts chart in IE11 + compatibility mode (IE 7 and 5) a critical error is thrown on point hover. _defaultGetter get's stoke value that is an object and it doesn't support the regex check done later.

Live demo with steps to reproduce

<html> 
    <head> 
        <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> 
        <title>Highcharts Example</title> 
        <!-- 1. Add JavaScript in the head of the page --> 
        <script src="https://code.highcharts.com/highcharts.src.js"></script>
        <script src="https://code.highcharts.com/modules/oldie.src.js"></script>
        <script src="https://code.highcharts.com/modules/sankey.src.js"></script>
        <script src="https://code.highcharts.com/modules/exporting.src.js"></script>
    </head>

    <body>
        <div id="container" style="width: 600px; height: 400px;"></div>

        <script>
          Highcharts.chart('container', {
            series: [{
              data: [1,2,3]
            }]
          });
        </script>
    </body> 
</html>

Affected browser(s)

IE11 + compatibility mode (7 & 5)

@KacperMadej KacperMadej added the Bug label Feb 5, 2018

@TorsteinHonsi TorsteinHonsi closed this in 45ff17e Feb 12, 2018

@zlateskud

This comment has been minimized.

Show comment
Hide comment
@zlateskud

zlateskud Feb 12, 2018

The issue is present on Sankey Diagram

I attach the code

highcharts error

sankey_test.zip

zlateskud commented Feb 12, 2018

The issue is present on Sankey Diagram

I attach the code

highcharts error

sankey_test.zip
@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 12, 2018

Contributor

Hi @zlateskud - issue should be fixed on our master branch, could you test it? http://github.highcharts.com/highcharts.src.js
Contributor

pawelfus commented Feb 12, 2018

Hi @zlateskud - issue should be fixed on our master branch, could you test it? http://github.highcharts.com/highcharts.src.js
@zlateskud

This comment has been minimized.

Show comment
Hide comment
@zlateskud

zlateskud Feb 12, 2018

hi @pawelfus - we have same error with version from git

err

zlateskud commented Feb 12, 2018 •
edited

hi @pawelfus - we have same error with version from git

err
@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Feb 12, 2018

Contributor

@zlateskud
The oldie.js module should be loaded from the github.

        <script src="https://github.highcharts.com/highcharts.src.js"></script>
        <script src="https://github.highcharts.com/modules/oldie.src.js"></script>
        <script src="https://github.highcharts.com/modules/sankey.src.js"></script>
        <script src="https://github.highcharts.com/modules/exporting.src.js"></script>

I have tested this and the error is not visible.
Contributor

KacperMadej commented Feb 12, 2018

@zlateskud
The oldie.js module should be loaded from the github.

        <script src="https://github.highcharts.com/highcharts.src.js"></script>
        <script src="https://github.highcharts.com/modules/oldie.src.js"></script>
        <script src="https://github.highcharts.com/modules/sankey.src.js"></script>
        <script src="https://github.highcharts.com/modules/exporting.src.js"></script>

I have tested this and the error is not visible.
@zlateskud

This comment has been minimized.

Show comment
Hide comment
@zlateskud

zlateskud Feb 12, 2018

@KacperMadej
Hi,
Hover is ok, thanks..
I tried to click, but the event does not work.
Thanks.

err button

zlateskud commented Feb 12, 2018

@KacperMadej
Hi,
Hover is ok, thanks..
I tried to click, but the event does not work.
Thanks.

err button
@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Feb 12, 2018

Contributor

Ok, but this is a different error. New issue's here: #7850
Contributor

KacperMadej commented Feb 12, 2018 •
edited

Ok, but this is a different error. New issue's here: #7850
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment