Skip to content

UI tweaks to cell creation#1192

Open
kasbah wants to merge 4 commits intomainfrom
kb/no-new-empty-cell
Open

UI tweaks to cell creation#1192
kasbah wants to merge 4 commits intomainfrom
kb/no-new-empty-cell

Conversation

@kasbah
Copy link
Copy Markdown
Member

@kasbah kasbah commented Apr 7, 2026

  • Doesn't create an empty "stem" cell when you unfocus without selecting anything
  • Newly created cells grab focus

@kasbah kasbah added frontend TypeScript frontend and Rust-wasm integrations ui/ux User interface and user experience labels Apr 7, 2026
@kasbah kasbah force-pushed the kb/no-new-empty-cell branch 2 times, most recently from 5808b94 to 038086c Compare April 10, 2026 14:19
Copy link
Copy Markdown
Member

@epatters epatters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm afraid this one doesn't seem to work properly for me (running frontend locally, on Firefox, rebased off main). When I press Shift+Enter with the active cell a stock at the top of a stock-flow notebook, the cell constructor menu appears at the end of the notebook.

Beyond that, it seems that this PR effectively removes stem cells from the notebook in that it is no longer possible to create them. I'm not committed to the stem cell concept (which was modeled on Notion). However, if we are going to replace it, we should go further and remove it as fully as practical from the codebase, though a complete removal would require a migration filtering notebook cells, which is annoying. In its current form, this PR seems to leave significant vestigial code.

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

Labels

frontend TypeScript frontend and Rust-wasm integrations ui/ux User interface and user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants