#23947 closed defect (bug) (invalid)
Twenty Thirteen: Chats not displaying correctly
Reported by: | Ipstenu | Owned by: | |
---|---|---|---|
Milestone: | Priority: | low | |
Severity: | normal | Version: | 3.6 |
Component: | Bundled Theme | Keywords: | |
Focuses: | Cc: |
Description (last modified by )
Reported by SomegirlwithA.com - http://wordpress.org/support/topic/chat-post-formatting
Basically the original twentythirteen showed chats like this:
speaker: blah blah blah
But now it's like this:
speaker: blah blah blah
ALSO if your speaker has two names, it totally barfs.
I copied the text from here: http://twentythirteendemo.wordpress.com/2013/02/11/eleven/
to my site running the latest SVN and got this:
Change History (11)
#3
@
12 years ago
- Summary changed from TwentyThirteen: Chats not displaying correctly to Twenty Thirteen: Chats not displaying correctly
#5
@
12 years ago
- Keywords reporter-feedback added
- Milestone changed from Awaiting Review to 3.6
- Priority changed from normal to low
- Severity changed from major to normal
#6
@
12 years ago
- Keywords reporter-feedback removed
@lancewillet - Good to know!
The chat text I used was this:
Nigel Tufnel: The numbers all go to eleven. Look, right across the board, eleven, eleven, eleven and… Marti DiBergi: Oh, I see. And most amps go up to ten? Nigel Tufnel: Exactly. Marti DiBergi: Does that mean it’s louder? Is it any louder? Nigel Tufnel: Well, it’s one louder, isn’t it? It’s not ten. You see, most blokes, you know, will be playing at ten. You’re on ten here, all the way up, all the way up, all the way up, you’re on ten on your guitar. Where can you go from there? Where? Marti DiBergi: I don’t know. Nigel Tufnel: Nowhere. Exactly. What we do is, if we need that extra push over the cliff, you know what we do? Marti DiBergi: Put it up to eleven. Nigel Tufnel: Eleven. Exactly. One louder. Marti DiBergi: Why don’t you just make ten louder and make ten be the top number and make that a little louder? Nigel Tufnel: These go to eleven.
I just pasted that in direct into plain HTML editor and it comes out blank. If I merge the names into one-word, then they show.
#7
@
12 years ago
Thanks—I can repeat it in trunk, too.
Just to be clear though, this isn't theme-related. Twenty Thirteen only styles the core HTML output.
Best to raise the issue on the core chat post format ticket instead: #23625.
#8
@
12 years ago
It wouldn't be the worst thing in the world if 23625-tests.diff got committed to Unit Tests since 1) I wrote them weeks ago and 2) that exact chat is one of the assertions
The tests were passing when the original code was committed, not sure what if anything got borked in subsequent commits
The WordPress.com demo site *is not* up to date with Post Formats, so please don't use it as a rubric until we merge 3.6 beta there.
The two-line display is intentional, see #23869 for how it is intended to look.
@ipstenu Can you paste here the chat text to test with for the two names issue?