Enable cell insertion without kernel and improve initial messages #419
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
goanpeca
force-pushed
the
enh/lexical-ui
branch
from
4b4e4ce to
9d91b92
Compare
goanpeca
changed the title
goanpeca
force-pushed
the
enh/lexical-ui
branch
from
9d91b92 to
fa8551e
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR enables cell insertion without a kernel and improves initial user messaging for better UX when no runtime is available. The changes focus on making the Jupyter notebook interface more graceful when operating without a connected kernel.
- Allow Jupyter cell creation and insertion even when no kernel is connected
- Replace generic error messages with user-friendly warnings about kernel connectivity
- Improve initial placeholder messages with better styling and clearer instructions
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
packages/react/src/components/output/OutputRenderer.tsx |
Adds ANSI-to-HTML conversion for colored error output display |
packages/react/src/components/output/OutputAdapter.ts |
Adds setOutputs method for programmatically updating output model |
packages/react/src/components/output/Output.tsx |
Improves output synchronization and Lumino widget updates |
packages/react/package.json |
Adds ansi-to-html dependency for error output formatting |
packages/lexical/src/plugins/JupyterInputOutputPlugin.tsx |
Removes kernel requirement for cell insertion and improves user messaging |
packages/lexical/src/plugins/ComponentPickerMenuPlugin.tsx |
Enables Jupyter cell option regardless of kernel availability |
packages/lexical/src/nodes/JupyterOutputNode.tsx |
Replaces console errors with user-facing warning messages |
packages/lexical/src/examples/index.tsx |
Switches default example to AppSimple |
packages/lexical/src/examples/ManualKernelControl.tsx |
Adds comprehensive example for manual kernel control |
packages/lexical/src/examples/AppSimple.tsx |
Adds kernel toggle functionality for testing |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
goanpeca
force-pushed
the
enh/lexical-ui
branch
from
fa8551e to
c0f29cf
Compare
goanpeca
force-pushed
the
enh/lexical-ui
branch
from
46c5063 to
7fa326c
Compare
echarles
approved these changes
Oct 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes datalayer/vscode-datalayer#34
Enable cell insertion without kernel and improve initial messages