RESOLVED FIXED 12317
webkit-block-placeholder class on placeholders seems unnecessary 
https://bugs.webkit.org/show_bug.cgi?id=12317
Summary webkit-block-placeholder class on placeholders seems unnecessary
Dan Wood
Reported 2007-01-18 11:56:00 PST
To reproduce: 0. Launch Blot_editableDiv_showSource, my modified version of "Blot" available from <http://dan.karelia.com/webkit/Blot_editableDiv_showSource.zip> with DYLD_FRAMEWORK_PATH pointing to TOT. (I'm using r18941) 1. Click in the top panel. 2. Type "Hello" and hit the return key Results: The contents of the editable DIV are: Hello.<div><br class="webkit-block-placeholder"></div> Expected: There shouldn't be an internal webkit class. Also the empty divs surrounding a <br> are weird -- I would expect there to be something like just a <br> but not enclosed in a div.
Attachments
Add attachment proposed patch, testcase, etc.
Dan Wood
Comment 1 2007-01-18 12:24:59 PST
Note also that the DIV does not self-close, it's not proper XHTML
Justin Garcia
Comment 2 2007-03-06 22:38:53 PST
If you want <br>s on carriage returns you could override insertNewline: and insertParagraphSeparator: and call insertLineBreak:, or you could use custom keybindings in your app. I'm going to make this bug about the seemingly unnecessary -webkit-block-placeholder class on the br. Please file a separate bug for the fact that the <br>s aren't self closing.
Mark Rowe (bdash)
Comment 3 2008-01-12 19:02:18 PST
<rdar://problem/5685601>
Justin Garcia
Comment 4 2008-02-05 13:56:09 PST
r30016
Note You need to log in before you can comment on or make changes to this bug.