feat(deps): upgrade BlockNote from 0.47.3 to 0.48.1#2224
Conversation
| /** | ||
| * Token Mantime | ||
| */ | ||
| & > .bn-container { |
There was a problem hiding this comment.
@nperez0111 Can you validate this change? 🙏
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThis pull request updates several Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Additional contextLinked issue 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Upgrades BlockNote dependencies to 0.48.1, adapts editor theming CSS for the .bn-root selector change, and relies on the upstream DOCX exporter fix to stop forcing an English spellchecker by default.
Changes:
- Bumped 9
@blocknote/*packages from0.47.3→0.48.1in the Impress app workspace. - Updated editor theming selector from
.bn-containerto.bn-rootper BlockNote0.48.0breaking change. - Refreshed
yarn.lockfor updated BlockNote (and related) dependency graph.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/frontend/apps/impress/package.json |
Updates BlockNote package versions to 0.48.1 for the app workspace. |
src/frontend/apps/impress/src/features/docs/doc-editor/styles.tsx |
Updates theming CSS selector to target .bn-root. |
src/frontend/yarn.lock |
Lockfile updates for BlockNote 0.48.1 and transitive dependency changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -15,7 +15,7 @@ export const cssEditor = css` | |||
| /** | |||
| * Token Mantime | |||
There was a problem hiding this comment.
The block comment says "Token Mantime"; this looks like a typo for "Mantine" (the rest of the file targets Mantine classes). Renaming it would avoid confusion when searching for Mantine-related styling.
| * Token Mantime | |
| * Token Mantine |
96af403 to
3ef6306
Compare
Bump all 11 @blocknote/* packages across impress and y-provider. Migrate theme CSS variables from .bn-container to .bn-root (0.48.0 breaking change). Fix comment typo: Mantime → Mantine. Fixes suitenumerique#1934 Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
3ef6306 to
f308af6
Compare
|
hey @StephanMeijer |
Ok! |
Summary
@blocknote/*packages from0.47.3to0.48.1.bn-containerto.bn-rootto handle the 0.48.0 theming breaking changeWhat changed
package.jsonAll 9
@blocknote/*packages bumped0.47.3→0.48.1:@blocknote/core,@blocknote/react,@blocknote/mantine,@blocknote/code-block,@blocknote/xl-ai,@blocknote/xl-docx-exporter,@blocknote/xl-multi-column,@blocknote/xl-odt-exporter,@blocknote/xl-pdf-exporterstyles.tsx(1 line)BlockNote 0.48.0 breaking change: theme CSS variables (colors, fonts) must now target
.bn-rootinstead of.bn-container..bn-rootwraps both the editor and its floating UI (menus, toolbars);.bn-containeris now layout-only.Fix for #1934
BlockNote 0.48.1 includes TypeCellOS/BlockNote#2651, which fixes the DOCX exporter defaulting to
en-USwhen no locale is passed. The<w:lang>element is now omitted entirely when no locale is provided, letting Word/LibreOffice auto-detect the document language from content — per ECMA-376 §17.3.2.20.Verification
yarn build— clean, no TypeScript errorsyarn test— 192/192 tests passyarn lint— clean@blocknote/xl-docx-exporter@0.48.1package sourceAssisted by Claude Sonnet 4.6