Add the "Remove formatting" functionality to rich-text editor

[limonte]

Currently, we don't have shortcuts for text alignment in rich-text editor.

But, when user paste text which was initially centered, the text alignment will be applied to squire as well. After that it's impossible to change the text alignment.

Steps to reproduce

1. Open Secure Compose window and click the "Send" button, the "Please add a recipient first" modal will be shown
2. Copy the "Please add a recipient first" text from the modal and close the modal
3. Make sure that the rich-text experimental feature is ON (it's available in Firefox only for now)
4. Paste the copied text to #input_text

Actual behavior

Expected behavior

[limonte]

@tomholub now that I see the screenshots, I'm thinking that we should also strip the pasted text from font-size, font-family, color styles. In other words, everything which we don't have controls for ATM. WDYT?

[tomholub]

I would instead suggest a "remove text formatting" button. On Gmail, I can select a bunch of text and remove formatting from it.

We do something similar on whole email - when I switch from rich text to simple text, we remove all formatting. So the difficult logic already exists. Just need a way to select part of the text, click the button which would cut it out, covert it to text, convert back to html, paste it back.

Maybe Squire already has some functionality for this. If not, we could do it the way I described. Does that sound ok?

It doesn't bother me that the user can paste things we don't support, as long as they have a way to remove it afterwards.

[limonte]

Sounds good! Actually, I like your suggestion better because it gives more flexibility to users.

Squire does have that functionality, it looks good:

• https://limonte.github.io/squire-sandbox/
• https://github.com/limonte/squire-sandbox/commit/29330611b12f92612fcfad886b3b874bd2981e46

For now we can make it work with shortcut only and later we can add the button, ATM I'm not sure what'd be the right place for the button.

---

Actually, I'd like to create a page with available FlowCrypt shortcuts, something similar to https://developers.google.com/web/tools/chrome-devtools/shortcuts

I guess the "Shortcuts" page would fit into the "Technical Docs" section

[tomholub]

yes - it would be great to have shortcuts documented!

[limonte]

@tomholub do I understand correctly that #2888 made https://github.com/flowcrypt/Squire obsolete? If so, we can consider removing or archiving that repo.

[limonte]

Implementing the shortcut (Ctrl/Cmd + \) is currently blocked by fastmail/Squire#401

[limonte]

Implementing the shortcut (Ctrl/Cmd + \) is currently blocked by neilj/Squire#401

Unblocked in #3378

[tomholub]

@tomholub do I understand correctly that #2888 made https://github.com/flowcrypt/Squire obsolete? If so, we can consider removing or archiving that repo.

Oh, that looks like an oversight! Did we fork it because Squire didn't want to merge that functionality in? It would be better to submit the links commit upstream.

[limonte]

@tomholub do I understand correctly that #2888 made https://github.com/flowcrypt/Squire obsolete? If so, we can consider removing or archiving that repo.

Oh, that looks like an oversight! Did we fork it because Squire didn't want to merge that functionality in? It would be better to submit the links commit upstream.

The changes were merged long ago to the upstream: https://github.com/neilj/Squire/commits/master?after=df2201038b77905ef588d24097a746ed8708617e+34&branch=master

We are already using the version from the upstream, installed via npm.

[limonte]

There was a delay in maintaining of the upstream, that's why we forked.

[tomholub]

Got it. In that case, I'll take the repo down. Thanks for mentioning this!