Skip to content

Finalize the PG CodeMirror Editor for the 2.20 release.#2719

Merged
pstaabp merged 2 commits into
openwebwork:WeBWorK-2.20from
drgrice1:freeze-pg-codemirror-editor-for-release
May 13, 2025
Merged

Finalize the PG CodeMirror Editor for the 2.20 release.#2719
pstaabp merged 2 commits into
openwebwork:WeBWorK-2.20from
drgrice1:freeze-pg-codemirror-editor-for-release

Conversation

@drgrice1

Copy link
Copy Markdown
Member

A non beta version 0.0.1 of the @openwebwork/pg-codemirror-editor has now been published. So make that the version for the 2.20 release. If there are any other changes the version can still be bumped if needed, but I doubt that there will be at this point.

The @openwebwork/codemirror-lang-pg package that the @openwebwork/pg-codemirror-editor depends on has a non beta version published (version 0.0.1).

The codemirror-lang-perl and codemirror-lang-mt packages are now also published in non beta status. Those are under my name still at this point since they are not specific to the openwebwork project.

@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch 2 times, most recently from 49e41b0 to 4357265 Compare April 27, 2025 16:22
@drgrice1 drgrice1 changed the base branch from develop to WeBWorK-2.20 April 29, 2025 12:20
@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch from 4357265 to 21bb64a Compare April 29, 2025 20:14
@drgrice1

drgrice1 commented May 5, 2025

Copy link
Copy Markdown
Member Author

This now also includes a change that fixes issue #2724.

@pstaabp pstaabp linked an issue May 6, 2025 that may be closed by this pull request
@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch from 707b37f to b2d19c7 Compare May 6, 2025 20:47
@somiaj

somiaj commented May 10, 2025

Copy link
Copy Markdown
Contributor

In testing if the spell check selection is saved, I noticed that the checkbox state is saved between page loads, but when loading a new problem, the spell checker checkbox is checked, but the spell checker is not working. I have to click the spell checker twice to get it working (one to uncheck the second to check) again.

@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch from b2d19c7 to f1dbcd3 Compare May 10, 2025 18:07
@drgrice1

Copy link
Copy Markdown
Member Author

@somiaj: I made another change that fixes that.

@somiaj somiaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Confirmed your change fixed the spell check issue.

@somiaj

somiaj commented May 10, 2025

Copy link
Copy Markdown
Contributor

@drgrice1 should we wait until you look into the PGML syntax highlighting issue I found before merging this?

@drgrice1

Copy link
Copy Markdown
Member Author

I think it would be good to go ahead and merge this. I am not sure when I will get to that issue.

@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch from f1dbcd3 to 0cff147 Compare May 10, 2025 19:23
@drgrice1

Copy link
Copy Markdown
Member Author

Nevermind. The version this points to now also fixes issue the PGML syntax highlighting issue.

@somiaj

somiaj commented May 10, 2025

Copy link
Copy Markdown
Contributor

Confirmed this fixed #2727

drgrice1 added 2 commits May 13, 2025 09:38
A non beta version 0.0.1 of the `@openwebwork/pg-codemirror-editor` has
now been published.  So make that the version for the 2.20 release.  If
there are any other changes the version can still be bumped if needed,
but I doubt that there will be at this point.

The `@openwebwork/codemirror-lang-pg` package that the
`@openwebwork/pg-codemirror-editor` depends on has a non beta version
published (version 0.0.1).

The `codemirror-lang-perl` and `codemirror-lang-mt` packages are now
also published in non beta status. Those are under my name still at this
point since they are not specific to the openwebwork project.
@drgrice1 drgrice1 force-pushed the freeze-pg-codemirror-editor-for-release branch from 0cff147 to 31188a0 Compare May 13, 2025 14:39
@pstaabp pstaabp merged commit 252a36a into openwebwork:WeBWorK-2.20 May 13, 2025
2 checks passed
@drgrice1 drgrice1 deleted the freeze-pg-codemirror-editor-for-release branch May 20, 2025 20:44
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.

pgProblemEditor doesn't save enable spell checking setting in develop.

3 participants