Skip to content

Conversation

@sofietoft
Copy link
Contributor

Description

Umbraco 15 introduces a new Rich Text Editor UI, Tiptap.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

15.0.0

Deadline (if relevant)

November 14th, or anytime before.

@sofietoft sofietoft marked this pull request as ready for review November 6, 2024 11:05
Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Looks good to me, @sofietoft 🙌 Just some minor suggestions w.r.t grammar, spellings. Nothing major 😀

This article is a work in progress and may undergo further revisions, updates, or amendments. The information contained herein is subject to change without notice.
{% endhint %}

The Rich Text Editor (RTE) is based on [tinymce](https://www.tinymce.com/) and is highly configurable. Depending on the configuration, it will give your content editors more flexibility when working with content that should be more than only plain text.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The Rich Text Editor (RTE) is based on [tinymce](https://www.tinymce.com/) and is highly configurable. Depending on the configuration, it will give your content editors more flexibility when working with content that should be more than only plain text.
The Rich Text Editor (RTE) is highly configurable and based on [TinyMCE](https://www.tinymce.com/). Depending on the configuration, it will give your content editors more flexibility when working with content that should be more than plain text.

@sofietoft
Copy link
Contributor Author

Thanks for the review @eshanrnh !

I've made some changes based on your suggestions - all images and links should be correct now.
And I've changed the wording a bit in the article about swapping the property editors.

Let me know what you think!

Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Looks good to me, @sofietoft 🙌 Just noticed some more things while previewing the PR.

# Styles

It is possible to define specific styles and fonts for the Rich Text Editor (RTE). Once you have defined the styles, and enabled them on the RTE Data Type, the styles can be accessed directly in the Content section.

Copy link
Contributor

Choose a reason for hiding this comment

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

This image is not rendering in the preview.

@sofietoft
Copy link
Contributor Author

Thanks again @eshanrnh 💪

Now I've finally caught everything.
In the configuration article I moved the list above the image, and updated the text before it slightly.
Let me know what you think! 🤞

@eshanrnh
Copy link
Contributor

eshanrnh commented Nov 7, 2024

Looks perfect 🙌 Thanks for applying the suggestions 💪

@sofietoft sofietoft merged commit 8aed0d3 into main Nov 7, 2024
19 checks passed
@sofietoft sofietoft deleted the 15/newRteUi branch November 28, 2024 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants