Skip to content

Commit 34b8cde

Browse files
RichDom2185sayomaki
authored andcommitted
Upgrade to Blueprint v5 (source-academy#2718)
* Replace deprecated `ITreeNode` with `TreeNodeInfo` Done as per migration guide. * Upgrade BlueprintJS dependencies to v5 * Use public `IconName` type instead of internal API Fixes a compile error following the version upgrade. * Migrate to `Datetime2` package Resolves the issue with the deprecated `DateInput` component. Also made the necessary API changes for the migration. * Update prop name Fixes the breaking change introduced by the Blueprint v5 upgrade. * Update test snapshots * Reran `yarn install` post-merge * Update snapshots post-merge * Update SCSS namespace * Fix popover styles Update the target classes to match the new components. * Revert Env Vizualiser snapshot change * Fix TreeNode comparison to compare id instead of object reference --------- Co-authored-by: sayomaki <sayomayomaki@gmail.com>
1 parent 138b215 commit 34b8cde

File tree

35 files changed

+2079
-2994
lines changed

35 files changed

+2079
-2994
lines changed

package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@
2626
"prepare": "husky"
2727
},
2828
"dependencies": {
29-
"@blueprintjs/core": "^4.20.1",
30-
"@blueprintjs/datetime": "^4.4.37",
31-
"@blueprintjs/icons": "^4.16.0",
32-
"@blueprintjs/popover2": "^1.14.11",
33-
"@blueprintjs/select": "^4.9.24",
29+
"@blueprintjs/core": "^5.7.0",
30+
"@blueprintjs/datetime2": "^2.2.7",
31+
"@blueprintjs/icons": "^5.5.0",
32+
"@blueprintjs/popover2": "^2.0.0",
33+
"@blueprintjs/select": "^5.0.0",
3434
"@octokit/rest": "^20.0.0",
3535
"@sentry/browser": "^7.57.0",
3636
"@sourceacademy/sharedb-ace": "^2.0.2",

src/commons/__tests__/__snapshots__/ContentDisplay.tsx.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ exports[`ContentDisplay page renders correctly 1`] = `
88
className="col-md-10 col-xs-11 contentdisplay-content-parent"
99
>
1010
<div
11-
className="bp4-card bp4-elevation-3 contentdisplay-content"
11+
className="contentdisplay-content bp5-card bp5-elevation-3"
1212
>
1313
<div>
1414
Test Content

src/commons/__tests__/__snapshots__/Markdown.tsx.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
exports[`Markdown page renders correctly 1`] = `
44
<div
5-
className="md bp4-running-text"
5+
className="md bp5-running-text"
66
dangerouslySetInnerHTML={
77
Object {
88
"__html": "<p>Welcome to the Source Academy playground!</p>

src/commons/assessment/__tests__/__snapshots__/Assessment.tsx.snap

Lines changed: 150 additions & 357 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)