Not possible to edit rich text fields (TinyMCE) such as submission title and abstract

[saalam]

Describe the bug
Title Field is locked and can't type anything when I try to insert title
To Reproduce
Steps to reproduce the behavior:

1. Go to 'Submissions'
2. Click on 'New Submission'
3. Scroll down to 'Title'
4. See error

What application are you using?
OJS version 3.4.0 PHP version 8.0.28

Additional information

[Vitaliy-1]

Hi @saalam,

Are there any related errors in the browser console?

[NateWr]

Yes, I strongly suspect this is a problem with the TinyMCE editor. It could be the built JavaScript in the release package or something. The error will most likely be detectable through an error in the browser's console.

There are several related issues. I'm going to keep this and close the others.

[NateWr]

@saalam (and anyone else experiencing this issue) can you also post details about your OS (Win/Mac/Linux) and browser (Chrome/Firefox/Edge)?

[saalam]

Hi @Vitaliy-1 To be honest I went through only PHP error log and I saw nothing about it in the error log file.
Hi @NateWr Regarding your question I'm using Windows OS with Chrome Browser.

[NateWr]

Looks like this is a build error. @asmecher I didn't realize that lib/ui-library was excluded from the release package. With the latest updates to TinyMCE, I chose to house some files in lib/ui-library/public/styles/tinymce/ to prevent duplication.

In hindsight, the decision to exclude lib/ui-library from the release package seems more sensible. In the PRs below, I've added a step to webpack so that it will copy the files from the tinymce directory in ui-library into lib/pkp/styles/tinymce. Then the applications load them from there, rather than the UI Library.

This fixed the problem for me, but can you test a release package?

One last decision to be made: these files will be copied whenever someone runs npm run dev or npm run build. In my view, these are build files, so we shouldn't commit them. That way they are always built from the correct branch of lib/ui-library. If you're happy with that (rather than committing them), let me know and I'll add this directory to the .gitignore file for each app.

PRs:
#8904
pkp/ojs#3871
pkp/omp#1388
pkp/ops#512

[asmecher]

@NateWr, those are sensible decisions. If you're ready to merge these, please go ahead; I'll build a new RC3 package and test it manually before making it available.

[NateWr]

Great! Merged those and added the directory to .gitignore in pkp-lib.