Mail: Sending mail inserts newlines

[jancborchardt]

Write a mail:

⏮ Last week:
- More 18 reviews and polishing
- Went through notifications

Send it, and see it formatted like this:

⏮ Last week:

- More 18 reviews and polishing

- Went through notifications

Priority cause it changes mail text unexpectedly. :)

[ChristophWurst]

The "problem"

• CKeditor uses paragraphs for Enter and line breaks for Shift+Enter
• Nextcloud changes paragraph styling to be indistinguishable with line breaks
  
  
  
  • This makes paragraphs in the editor look like line breaks
  
  
• When you view your sent email you see actual paragraph formatting because the iframe does not use Nextcloud styling

Priority cause it changes mail text unexpectedly. :)

Yeah, pretty much every bug report is about unexpected behavior :see_no_evil:

[ChristophWurst]

Regarding <p> vs <br> https://github.com/ckeditor/ckeditor5/issues/617

[jancborchardt]

Yeah, pretty much every bug report is about unexpected behavior see_no_evil

Let me say it like this: Priority because it not only is a bug which trips you up, but makes you look like an idiot to people receiving your emails. ;)

[StCyr]

Wouldn't it be possible to send Nexcloud's p style together with the email?

Like adding style="margin:0;" to all <p> tags?

[Friptick]

The guy at CKEditor is right, line breaks are the root of all evil, visually and semantically. They must be discouraged at all costs.

[StCyr]

Good. Now, what’s the solution to get the emails properly formated?

Le 19 janv. 2020 à 10:22, Friptick [e-mail] a écrit :


The guy at CKEditor is right, line breaks are the root of all evil, visually and semantically. They must be discouraged at all costs.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

[ChristophWurst]

I'd be fine with a custom styling on the p's right now.

[jancborchardt]

Reopening since this still happens – ref @ma12-co if you have additional logs since you ran into that issue recently.

[ma12-co]

Yep, still happening as of today on our company instance

[StCyr]

Isn't it solved now? @ma12-co ?

[ma12-co]

It didn't happen to me recently, but I haven't actively tried to reproduce. Do you need help with that?

[StCyr]

It didn't happen to me recently, but I haven't actively tried to reproduce. Do you need help with that?

Well, at least I don't remember how to reproduce this issue 😞

[cwx0046]

How about adding an option that makes Enter insert <p> tag with mb-0 style:

```javascript
editorConfig: {
emailMode: true
}
````

```html

````

[ChristophWurst]

emailMode? Is that a feature of CKEditor 5?