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

TypeError: Attempted to assign to readonly property #19516

Closed
abaluta opened this issue Aug 8, 2023 · 3 comments · Fixed by #19594
Closed

TypeError: Attempted to assign to readonly property #19516

abaluta opened this issue Aug 8, 2023 · 3 comments · Fixed by #19594

Comments

@abaluta
Copy link

abaluta commented Aug 8, 2023

Expected behaviour

No errors

Actual behaviour

When the option styledMode is true
JavaScript error: TypeError: Attempted to assign to readonly property.

Live demo with steps to reproduce

https://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/css/prefers-color-scheme
Enable or disable charts: {styledMode: true|false}

Product version

Highcharts

Affected browser(s)

Safari (Version 16.6 (18615.3.12.11.2))

Discussion of the error on the forum

https://www.highcharts.com/forum/viewtopic.php?f=12&t=50680&sid=9eb5202e6c8ab7f3fde9e4279f785e88

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Aug 8, 2023
@karolkolodziej
Copy link
Contributor

I can reproduce this issue only in Safari.

Looks like it was introduced in v10.1.0 but I was unable to bissect.
Needs further investigation.

@Samueloyeks
Copy link

I also have the same issue on safari

@hubertkozik hubertkozik self-assigned this Aug 18, 2023
@hubertkozik
Copy link
Member

I have prepared a workaround for this bug. You can find it in the demo below. If you want to use it in your project for now you can just copy-paste the IIFE function.

Live demo: https://jsfiddle.net/BlackLabel/9yL3qkg0/

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

Successfully merging a pull request may close this issue.

5 participants