Skip to content

fix: fix prosemirror version mismatches#1242

Merged
3mcd merged 2 commits into
em/pm-tablesfrom
tfk/fix-table-versions
May 13, 2025
Merged

fix: fix prosemirror version mismatches#1242
3mcd merged 2 commits into
em/pm-tablesfrom
tfk/fix-table-versions

Conversation

@tefkah

@tefkah tefkah commented May 13, 2025

Copy link
Copy Markdown
Member

Issue(s) Resolved

ProseMirror versions have somewhat diverged. this would cause an issue when inserting math in tables because you would get a ProseMirror model is not the same error. this PR fixes that by updating most of our prosemirror-related packages to the latest versions and pinning the most variable versions in the catalog. This includes @handlewithcare/react-prosemirror which should fix this issue handlewithcarecollective/react-prosemirror#51 raised by @allisonking

High-level Explanation of PR

Test Plan

Screenshots (if applicable)

Notes

@tefkah tefkah requested a review from 3mcd May 13, 2025 12:07
@3mcd 3mcd merged commit b29573b into em/pm-tables May 13, 2025
15 of 16 checks passed
@3mcd 3mcd deleted the tfk/fix-table-versions branch May 13, 2025 14:26
3mcd added a commit that referenced this pull request May 15, 2025
* feat: first attempt at adding prosemirror-tables

* feat: add columnResizing plugin

* chore: more table work

* feat: make tables prettier

* feat: remove alternating background

* chore: bump prosemirror view

* feat: get tabbing working

* feat: add figure

* feat: make it look a lil more like allison's

* chore: downgrade prosemirror-view

* feat: toggle function

* keep panel open when title/caption toggled

* get marks working

* simplify

* simplify

* css

* fix up media upload

* credit and license

* fix type error

* remove unused code

* remove more unused code

* dont open by default

* fix: fix some node playwright tests

* fix: make the media playwright tests fail later

* a little cleanup

* restore closing behavior

* fix: fix prosemirror version mismatches (#1242)

* fix: fix prosemirror version mismatches

* chore: remove ts-expect-error

* fix type error

* more cleanup

* typerror

* fix toggle test

* table resizing works now yay

* fix key issue

* try testid

* try blur

* open advanced

* expand advanced options

* fix mark attr updates

* resolve closest node to cursor for attributepanel

* try this

* force test run

* active node is node after from and node before to

* try state.selection. instead of anchor

* fix a couple more tests

* try this

* try doubleclick

* dblclick

* try dblclick again

* nullish form control

* redundant click

* remove id and class from MediaUpload menu

* add form

* Update packages/context-editor/src/style.css

Co-authored-by: Thomas F. K. Jorna <hello@tefkah.com>

* Update packages/context-editor/src/style.css

Co-authored-by: Thomas F. K. Jorna <hello@tefkah.com>

* Update packages/context-editor/src/style.css

Co-authored-by: Thomas F. K. Jorna <hello@tefkah.com>

---------

Co-authored-by: Thomas F. K. Jorna <hello@tefkah.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants