Skip to content

TinyMCE toolbar failed to display, needs to adapt to new name, When upgrading to 6. x #7564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
ahjszll opened this issue Feb 10, 2025 · 3 comments · May be fixed by #7672
Open

TinyMCE toolbar failed to display, needs to adapt to new name, When upgrading to 6. x #7564

ahjszll opened this issue Feb 10, 2025 · 3 comments · May be fixed by #7672
Assignees
@DmitriyKulagin
Labels
refactoring / source code
Milestone
Version 4.90

Comments

@ahjszll
Copy link

ahjszll commented Feb 10, 2025

reference address
https://www.tiny.cloud/docs/tinymce/6/migration-from-5x/#things-we-renamed

src/Presentation/Nop.Web/Areas/Admin/Views/Shared/EditorTemplates/RichEditor.cshtml

Line:92

error code
toolbar: "ltr rtl | insertfile undo redo | styleselect | fontselect | fontsizeselect | bold italic | forecolor backcolor | alignleft aligncenter alignright alignjustify | bullist numlist outdent indent | link image",

Correct code
toolbar: "ltr rtl | insertfile undo redo | styles | fontfamily | fontsize | bold italic | forecolor backcolor | alignleft aligncenter alignright alignjustify | bullist numlist outdent indent | link image",
@G3r4rd00
Copy link

@DmitriyKulagin
Hello, I have a solution for this issue. Can I submit a PR to fix it, or is something already in progress?

Image

@DmitriyKulagin
Copy link
Contributor

@G3r4rd00
Hi, You can do PR.

@AndreiMaz AndreiMaz added this to the Version 4.90 milestone Apr 8, 2025
@KaranChadha10 KaranChadha10 linked a pull request Apr 9, 2025 that will close this issue
Open
@KaranChadha10
Copy link
Contributor

I've created a PR (#7672) to address this issue. Please review when possible. Thanks!

FYI: @DmitriyKulagin @AndreiMaz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DmitriyKulagin DmitriyKulagin

Labels
refactoring / source code
Projects
None yet
Milestone
Version 4.90
Development

Successfully merging a pull request may close this issue.

fix: [7654] update TinyMCE toolbar for v6 compatibility KaranChadha10/nopCommerce
5 participants
@ahjszll @G3r4rd00 @AndreiMaz @DmitriyKulagin @KaranChadha10