|Summary:||Whitespace in particular source code changes rendering; does not in Firefox|
|Product:||WebKit||Reporter:||Lucas Buetto <citrin.sistemas>|
|Severity:||Normal||CC:||abarth, ap, a.renevier, arpitabahuguna, commit-queue, eric, esprehn+autocc, leviw, mitz, ojan.autocc, robert, webkit.review.bot|
|Version:||525.x (Safari 3.2)|
Description Lucas Buetto 2012-11-01 04:23:49 PDT
I found a problem that occurs only when I am using indentation of code. Turning the html code I am enclosing note that the 'first instance' icons are going to line the bottom of the div where he is. I noticed that this happens only when my code is indented as in the example below: <div class="ComboBox"> <span>System</ span> <span class="ComboIcon IconComboDown"></ span> </div> But in 'Second Example' the problem ceases to be because I removed the indentation of code, leaving him as the example below: <div class="ComboBox"><span>System</span><span class="ComboIcon IconComboDown"></span></div> User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.4 (KHTML, like Gecko) Safari/537.4 Chrome/22.0.1229.94. The problem occurs with the browsers: - Using Safari Version: 5.1.7 (7534.57.2). - Google Chrome 22.0.1229.94 (Official Build 161065) m
Comment 1 Adam Barth 2012-11-01 10:09:34 PDT
> <div class="ComboBox"> > <span>System</ span> > <span class="ComboIcon IconComboDown"></ span> > </div> Is there CSS that uses these class attributes? Also, you seem to have an extra space between the "</" and the "span" in this version but not in the other version. Is that important to the example?
Comment 2 Eric Seidel (no email) 2012-11-01 10:32:05 PDT
Created attachment 171889 [details] stand-alone reduction
Comment 3 Eric Seidel (no email) 2012-11-01 10:37:14 PDT
Yup, this is a layout bug, not a parser bug as far as I can tell.
Comment 4 Levi Weintraub 2012-11-01 10:51:04 PDT
Awaiting a final patch on the other bug. *** This bug has been marked as a duplicate of bug 99442 ***
Comment 5 Alexey Proskuryakov 2012-11-01 10:54:45 PDT
This bug is not a recent regression. Is it correct that it got marked as a duplicate of recent regression?
Comment 6 Eric Seidel (no email) 2012-11-01 11:00:51 PDT
I guess the question is if it is fixed by bug 99442 or not. I don't know the answer to that.
Comment 7 Levi Weintraub 2012-11-02 05:46:38 PDT
You're right in that this isn't a regression. My mistake marking it a duplicate, it's not.
Comment 9 Darin Adler 2013-04-09 09:20:33 PDT
Comment on attachment 192574 [details] patch proposal I think this is right. Would be even better if a true rendering expert reviewed it, but we can always roll it out if they disagree.
Comment 10 WebKit Commit Bot 2013-04-09 09:45:18 PDT
Comment on attachment 192574 [details] patch proposal Clearing flags on attachment: 192574 Committed r148027: <http://trac.webkit.org/changeset/148027>
Comment 11 WebKit Commit Bot 2013-04-09 09:45:22 PDT
All reviewed patches have been landed. Closing bug.