Skip to content

Comments

Polyfill Array.at() for Qt5 compatibility#501

Merged
jitseniesen merged 4 commits intospyder-ide:masterfrom
jitseniesen:markdown
Oct 15, 2025
Merged

Polyfill Array.at() for Qt5 compatibility#501
jitseniesen merged 4 commits intospyder-ide:masterfrom
jitseniesen:markdown

Conversation

@jitseniesen
Copy link
Member

@jitseniesen jitseniesen commented Oct 14, 2025

The internal console shows the error message

TypeError: t.at is not a function. Please report this to https://github.com/markedjs/marked

Probably related to this, markdown cells are not displayed correctly.

This looks to be related to issue markedjs/marked#3546, which is about the member function at() of the (standard library?) class Array. Apparently, this function is only defined in Chrome 92, but Spyder includes Qt WebEngine 5.15.15 which is based on Chromium 87. One fix would be to update to at least WebEngine 6.2.5 but that needs us to package Qt6 WebEngine.

This PR includes another fix: it uses a polyfill from core-js to define the missing function (if I understand everything correctly). This does seem to resolve the markdown issue. Additionally, the PR syncs some versions of JavaScript libraries to eliminate more warnings in the internal console.

Fixes #500.

Following Jupyter notebook PR #7735.
This is to follow Jupyter which also uses react version 18.2.0.
Follows Jupter notebook PR #7198 which was part of Jupyter
notebook version 7.1. This elimiates the "plugin xxx is already
registered" error messages.
@jitseniesen jitseniesen added this to the v0.7.1 milestone Oct 14, 2025
@jitseniesen jitseniesen self-assigned this Oct 14, 2025
@jitseniesen jitseniesen changed the title Polygill Array.at Polyfill Array.at() for Qt5 compatibility Oct 14, 2025
@jitseniesen jitseniesen marked this pull request as ready for review October 14, 2025 20:16
@jitseniesen jitseniesen merged commit 86c2604 into spyder-ide:master Oct 15, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown Cells Do Not Render

1 participant