Make WordPress Core

Opened 21 years ago

Closed 17 years ago

#2285 closed enhancement (wontfix)

Wpautop strips blank lines

Reported by: linickx Owned by: skeltoac
Priority: normal Milestone:
Component: Formatting Version: 2.0
Severity: normal Keywords:
Cc: Focuses:

Description

Non-Techie users like to user spaces & carrige returns for formatting text.

Using the WYSIWYG gui, if you use carrige return 3 times in a text - hoping to create some white space between text - i.e. like entering text1<br /><br /><br />text2 or <p>text1</p><p>&nbsp;</p><p>&nbsp;</p><p>&nbsp;</p><p>text2</p> - the gui removes them and replace with single <p>text1</p><p>text2</p> which is one line of white space.

I understand that this kind of use may not be best practice, but it's the kind of thing normal users do, and wp is supposed to be a good all round user experience.

Change History (20)

#1 @markjaquith
21 years ago

  • Owner matt removed

#2 follow-up: @davidhouse
21 years ago

Thanks Mark. I aksed him to do that: please don't assign tickets to people without their permission. There are more people than matt that can fix your bug. If matt decides he wants to fix it, he can accept it.

#3 @davidhouse
21 years ago

Now, for the fix, we could leave anything more than two line breaks as <br/>s instead of trying to paragraph them. We could do the same for more than one space: convert them all to &nbsp;.

#4 @skeltoac
21 years ago

  • Milestone → 2.1
  • Owner set to skeltoac
  • Severity major → enhancement
  • Status new → assigned

I'll name that tune in 2.1 notes. If somebody wants to name that tune in 2.0.1 notes, accept the ticket and run with it! :-)

#5 @linickx
21 years ago

thanks for taking this on guys :-D

[p.s. fyi:- not sure what the 1st comments are about, but new calls assign to matt if you leave the box blank :-) ]

#6 @matt
20 years ago

  • Milestone 2.1 → 2.2

#7 @foolswisdom
20 years ago

  • Milestone 2.2 → 2.4

#8 in reply to: ↑ 2 @rob1n
20 years ago

Replying to davidhouse:

Thanks Mark. I aksed him to do that: please don't assign tickets to people without their permission. There are more people than matt that can fix your bug. If matt decides he wants to fix it, he can accept it.

It automatically assigns it to Matt if it's in the WordPress.org component, I believe.

#9 @rob1n
20 years ago

  • Component WordPress.org → TinyMCE

#10 @Otto42
20 years ago

This is not a TinyMCE thing, it's wpautop that's doing it. And frankly, I think it's desired behavior. If they want vertical space, they should be using styles on the paragraph. Okay, not the best UX, but perhaps an easier way to insert a vertical space is called for instead of letting the user enter non-semantic P's and BR's?

#11 @thee17
19 years ago

  • Resolution → wontfix
  • Status assigned → closed

Suggest closing as this is a disired result rather than an error.

#12 @Nazgul
19 years ago

  • Milestone 2.5

#13 @lloydbudd
19 years ago

  • Component TinyMCE → General
  • Milestone → 2.6
  • Resolution wontfix
  • Status closed → reopened

Re-opening. You are thinking like developers not every day people. Originally this bug had some traction from Andy, and it still seems like the desirable behavior for most people, "I want more spaces on a line so I use the space bar"

#14 @thee17
19 years ago

I thought this was dealing with carrage returns as opposed to spaces. I could see leaving blank br's but not many blank p's.

#15 @lloydbudd
19 years ago

Sorry, you are correct about the issue described, but it doesn't change the user's expectations of it to just work.

#16 @lloydbudd
19 years ago

  • Keywords WYSIWYG user experience removed
  • Summary WYSIWYG strips white space - bad user experience → Editor strips blank lines

#17 @azaozz
19 years ago

  • Summary Editor strips blank lines → Wpautop strips blank lines

The blank lines are stripped by wpautop(), both the js and php versions of it. This seems to be related to users complaining that they cannot enter <br /> tags in a post, but actually trying to produce invalid XHTML (which, of course, is auto-corrected).

Perhaps wpautop should be changed to replace 3 consecutive <br /> with <p>&nbsp;</p>.

#18 @Denis-de-Bernardy
18 years ago

@azaozz: this wouldn't do much good if tinymce won't let a user insert a non-breaking space.

suggesting wontfix

#19 @Denis-de-Bernardy
17 years ago

  • Component General → Formatting

#20 @Denis-de-Bernardy
17 years ago

  • Milestone 2.9
  • Resolution → wontfix
  • Status reopened → closed

this has been rotting for long enough

Note: See TracTickets for help on using tickets.