VisualEditor: Move to using white-space: pre-wrap instead of using nbsp's to fake spaces
Closed, ResolvedPublic8 Estimated Story Points
Actions

Details

	Subject	Repo	Branch	Lines +/-
	Use white-space: pre-wrap instead of using   for spaces	VisualEditor/VisualEditor	master	+15 -74

		Status	Subtype	Assigned	Task
		Resolved		matmarex	T96487 In VisualEditor, footnote immediately following a period sometimes wraps to new line even though there is space left in the line
		Resolved		matmarex	T55931 VisualEditor: Move to using white-space: pre-wrap instead of using nbsp's to fake spaces

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:02 AM

• bzimport set Reference to bz53931.

From an email conversation I had with Aryeh in January 2012:

(Since I'm thinking about canonicalization of whitespace, a tip: set
white-space: pre-wrap on any contenteditable regions you use, and
wherever you use them. This will stop you from needing to use nbsp's
in ways they were never intended to be used, which causes a variety of
subtle bugs. See
http://dvcs.w3.org/hg/editing/raw-file/tip/editing.html#canonical-space-sequences
for a basic explanation, and click the first "View Comments" at
http://dvcs.w3.org/hg/editing/raw-file/tip/editing.html#the-inserttext-command
for a more detailed explanation of how nbsp substitution works in
practice.)

Still valid:

./src/ce/nodes/ve.ce.TextNode.js:		// Replace spaces with &nbsp; where needed

matmarex subscribed.

Change 255931 had a related patch set uploaded (by Bartosz Dziewoński):
Use white-space: pre-wrap instead of using   for spaces

Jdforrester-WMF updated the task description. (Show Details)

Jdforrester-WMF set Security to None.

Jdforrester-WMF edited a custom field.

Jdforrester-WMF updated the task description. (Show Details)

Change 255931 merged by jenkins-bot:
Use white-space: pre-wrap instead of using   for spaces