umbraco / Umbraco-CMS

Umbraco is a free and open source .NET content management system helping you deliver delightful digital experiences.
https://umbraco.com
MIT License
4.36k stars 2.64k forks source link

Template plugin no longer show up in richtext toolbar #16608

Open bjarnef opened 1 week ago

bjarnef commented 1 week ago

Which Umbraco version are you using? (Please write the exact version, example: 10.1.0)

13.3.2

Bug summary

In a Umbraco v12.3.10 we used Template plugin, which show up in richtext editor toolbar.

In Umbraco v13.3.2 we have enabled the Template plugin, which show up in configuration.

image

However it doesn't show up in richtext editor toolbar.

image

"RichTextEditor": {
  "CustomConfig": {
    "formats": "{\"cite\":{\"block\":\"cite\"}},\"checklist\":{\"block\":\"ul\",\"classes\":\"check-list\"}",
    "style_formats": "[{\"title\":\"Paragraph\",\"format\":\"p\"},{\"title\":\"Blockquote\",\"format\":\"blockquote\"},{\"title\":\"Cite\",\"format\":\"cite\"},{\"title\":\"Code\",\"format\":\"code\"},{\"title\":\"Headers\",\"items\":[{\"title\":\"Heading h1\",\"block\":\"h1\"},{\"title\":\"Heading h2\",\"block\":\"h2\"},{\"title\":\"Heading h3\",\"block\":\"h3\"},{\"title\":\"Heading h4\",\"block\":\"h4\"},{\"title\":\"Heading h5\",\"block\":\"h5\"},{\"title\":\"Heading h6\",\"block\":\"h6\"}]},{\"title\":\"Checkmark list ✓\",\"format\":\"checklist\"}]",
    "templates": "[{\"title\":\"Checkmark list\",\"description\":\"Unordnered list with checkmark bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-checkmark.html\"}]"
  },
  "Commands": [
    {
      "Alias": "blockquote",
      "Name": "Blockquote",
      "Mode": "Selection"
    },
    {
      "Alias": "template",
      "Name": "Template",
      "Mode": "All"
    }
  ]
}

Specifics

No response

Steps to reproduce

Enabled template plugin of richtext editor.

Expected result / actual result

No response

github-actions[bot] commented 1 week ago

Hi there @bjarnef!

Firstly, a big thank you for raising this issue. Every piece of feedback we receive helps us to make Umbraco better.

We really appreciate your patience while we wait for our team to have a look at this but we wanted to let you know that we see this and share with you the plan for what comes next.

We wish we could work with everyone directly and assess your issue immediately but we're in the fortunate position of having lots of contributions to work with and only a few humans who are able to do it. We are making progress though and in the meantime, we will keep you in the loop and let you know when we have any questions.

Thanks, from your friendly Umbraco GitHub bot :robot: :slightly_smiling_face:

bjarnef commented 1 week ago

I was a bit too fast here. It also need the actual template plugin enabled.

"RichTextEditor": {
  "CustomConfig": {
    "formats": "{\"cite\":{\"block\":\"cite\"}},\"checklist\":{\"block\":\"ul\",\"classes\":\"check-list\"}",
    "style_formats": "[{\"title\":\"Paragraph\",\"format\":\"p\"},{\"title\":\"Blockquote\",\"format\":\"blockquote\"},{\"title\":\"Cite\",\"format\":\"cite\"},{\"title\":\"Code\",\"format\":\"code\"},{\"title\":\"Headers\",\"items\":[{\"title\":\"Heading h1\",\"block\":\"h1\"},{\"title\":\"Heading h2\",\"block\":\"h2\"},{\"title\":\"Heading h3\",\"block\":\"h3\"},{\"title\":\"Heading h4\",\"block\":\"h4\"},{\"title\":\"Heading h5\",\"block\":\"h5\"},{\"title\":\"Heading h6\",\"block\":\"h6\"}]},{\"title\":\"Checkmark list ✓\",\"format\":\"checklist\"}]",
    "templates": "[{\"title\":\"Checkmark list\",\"description\":\"Unordnered list with checkmark bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-checkmark.html\"}]"
  },
  "Commands": [
    {
      "Alias": "blockquote",
      "Name": "Blockquote",
      "Mode": "Selection"
    },
    {
      "Alias": "template",
      "Name": "Template",
      "Mode": "All"
    }
  ],
  "Plugins": [
    "template"
  ]
}
iOvergaard commented 1 week ago

Hi @bjarnef, could this be related to #16602 ?

bjarnef commented 1 week ago

Hi @iOvergaard

It could be related, but actually I missed this part:

"Plugins": [
  "template"
]

image

However I am not sure how we can style the template HTML. I tried including link to rte.css, include a <style> block in <head> and <body>, but it seems TinyMCE strip this.

image

So visually it is not great to content editor as it show the default HTML rendered.

But I do get the rte.css when inserting template in editor.

image

However it does seems to mess up a bit with styled configured in appsettings.json.

image

bjarnef commented 1 week ago

These seems to be default though.

I also noticed the "Pre" style block has some scrollbar.

image

which it inherits from the global pre element style.

image

bjarnef commented 1 week ago

This configuration I have at the moment:

"RichTextEditor": {
  "CustomConfig": {
    "formats": "{\"cite\":{\"block\":\"cite\"}},\"checklist\":{\"block\":\"ul\",\"classes\":\"check-list\"},\"arrowlist\":{\"block\":\"ul\",\"classes\":\"arrow-list\"},\"numberlist\":{\"block\":\"ol\",\"classes\":\"number-list\"}",
    "style_formats": "[{\"title\":\"Paragraph\",\"format\":\"p\"},{\"title\":\"Blockquote\",\"format\":\"blockquote\"},{\"title\":\"Cite\",\"format\":\"cite\"},{\"title\":\"Code\",\"format\":\"code\"},{\"title\":\"Headers\",\"items\":[{\"title\":\"Heading h1\",\"block\":\"h1\"},{\"title\":\"Heading h2\",\"block\":\"h2\"},{\"title\":\"Heading h3\",\"block\":\"h3\"},{\"title\":\"Heading h4\",\"block\":\"h4\"},{\"title\":\"Heading h5\",\"block\":\"h5\"},{\"title\":\"Heading h6\",\"block\":\"h6\"}]},{\"title\":\"Checkmark list ✓\",\"format\":\"checklist\"},{\"title\":\"Arrow list →\",\"format\":\"arrowlist\"},\"format\":\"numberlist\"},{\"title\":\"Number list ➀\",\"format\":\"numberlist\"}]",
    "templates": "[{\"title\":\"Checkmark list\",\"description\":\"Unordnered list with checkmark bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-checkmark.html\"},{\"title\":\"Arrow list\",\"description\":\"Unordnered list with arrow bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-arrow.html\"},{\"title\":\"Number list\",\"description\":\"Ordnered list with numbers\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/ordered-list-number.html\"}]"
  },
  "Commands": [
    {
      "Alias": "blockquote",
      "Name": "Blockquote",
      "Mode": "Selection"
    },
    {
      "Alias": "template",
      "Name": "Template",
      "Mode": "All"
    }
  ],
  "Plugins": [
    "template"
  ]
},

Update: It has a syntax error. It is now this:

"RichTextEditor": {
  "CustomConfig": {
    "formats": "{\"cite\":{\"block\":\"cite\"}},\"checklist\":{\"block\":\"ul\",\"classes\":\"check-list\"},\"arrowlist\":{\"block\":\"ul\",\"classes\":\"arrow-list\"},\"numberlist\":{\"block\":\"ol\",\"classes\":\"number-list\"}",
    "style_formats": "[{\"title\":\"Paragraph\",\"format\":\"p\"},{\"title\":\"Blockquote\",\"format\":\"blockquote\"},{\"title\":\"Cite\",\"format\":\"cite\"},{\"title\":\"Code\",\"format\":\"code\"},{\"title\":\"Headers\",\"items\":[{\"title\":\"Heading h1\",\"block\":\"h1\"},{\"title\":\"Heading h2\",\"block\":\"h2\"},{\"title\":\"Heading h3\",\"block\":\"h3\"},{\"title\":\"Heading h4\",\"block\":\"h4\"},{\"title\":\"Heading h5\",\"block\":\"h5\"},{\"title\":\"Heading h6\",\"block\":\"h6\"}]},{\"title\":\"Checkmark list ✓\",\"format\":\"checklist\"},{\"title\":\"Arrow list →\",\"format\":\"arrowlist\"},{\"title\":\"Number list ➀\",\"format\":\"numberlist\"}]",
    "templates": "[{\"title\":\"Checkmark list\",\"description\":\"Unordnered list with checkmark bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-checkmark.html\"},{\"title\":\"Arrow list\",\"description\":\"Unordnered list with arrow bullets\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/unordered-list-arrow.html\"},{\"title\":\"Number list\",\"description\":\"Ordnered list with numbers\",\"url\":\"/App_Plugins/MCB.Umbraco.RTE/Templates/ordered-list-number.html\"}]"
  },
  "Commands": [
    {
      "Alias": "blockquote",
      "Name": "Blockquote",
      "Mode": "Selection"
    },
    {
      "Alias": "template",
      "Name": "Template",
      "Mode": "All"
    }
  ],
  "Plugins": [
    "template"
  ]
}

image

However it is odd these are disabled. I think they used to work and be able on ul, ol or depending on the selector allowing the format.