Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 877141 - Strip down non-break spaces from contenteditable #10333

Merged
merged 1 commit into from Jun 12, 2013

Conversation

lissyx
Copy link
Contributor

@lissyx lissyx commented Jun 12, 2013

When several consecutives spaces are typed in a contenteditable element,
all but the last one are converted to non-break spaces. We don't want
this, here, so we simply replace them with normal spaces.

@@ -190,6 +190,13 @@ var Compose = (function() {

var last = content.length - 1;
var text = node.textContent;

// Bug 877141 - contenteditable wil insert non-break spaces when
// multiple consecutive spaces are entered, we don't want of them.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: "we don't want them"

When several consecutives spaces are typed in a contenteditable element,
all but the last one are converted to non-break spaces. We don't want
this, here, so we simply replace them with normal spaces.
julienw added a commit that referenced this pull request Jun 12, 2013
Bug 877141 - Strip down non-break spaces from contenteditable

r=julien
@julienw julienw merged commit 5db84b0 into mozilla-b2g:master Jun 12, 2013
julienw added a commit that referenced this pull request Jun 13, 2013
Bug 877141 - Strip down non-break spaces from contenteditable

r=julien(cherry picked from commit 5db84b0)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants