search

openedx / frontend-lib-content-components

[Moved to openedx/frontend-app-course-authoring] A library of high-level components for content handling (viewing, editing, etc. of HTML, video, problems, etc.), to be shared by multiple MFEs.
GNU Affero General Public License v3.0
11 stars 33 forks source link

Text editor does not save paragraph break formatting when the table is followed by text #481

Closed mariajgrimaldi closed 5 months ago

mariajgrimaldi commented 5 months ago

Expected behavior

The saved version of the xblock should retain the same formatting Instructor created for the xblock. The text after the table should be displayed with the new paragraph. Text xblock editing

Actual behavior

The saved version does not retain the paragraph break after the table. The text added after the table appears immediately after the table without any paragraph break. student view Studio xblock view

Steps to reproduce

  1. In Studio select an option to edit Text xblock.
  2. Add a Table to the body of the Xblock.
  3. Add a paragraph after the table.
  4. Add text.
  5. save Xblock

Additional information

Tried adding multiple paragraph breaks and horizontal lines in between the table and the text, but the result was always the same - upon saving the xblock, the formatting is as in the screenshots.

Here's a course you can use for testing: course.kv5s4klf.tar.gz

Use the Text Xblocks section.

Original issue: https://github.com/openedx/wg-build-test-release/issues/370

mariajgrimaldi commented 5 months ago

FYI @arbrandes @brian-smith-tcril @openedx/2u-tnl, please tag anyone who can help.

mariajgrimaldi commented 5 months ago

I'm not sure if this is even an issue or simply a misuse of the TinyMCE editor, but I'm also opening the issue for documentation purposes.

mariajgrimaldi commented 5 months ago

I tested with the legacy editor, and it also happens:

Screencast from 02-06-24 12:24:25.webm

It looks like a weird formatting issue with the text editors. I'll close the issue for now. Thanks!

mariajgrimaldi commented 5 months ago

If you think this is something we can fix, please feel free to reopen the issue.