Summary: | Whitespace in particular source code changes rendering; does not in Firefox | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Lucas Buetto <citrin.sistemas> | ||||||
Component: | DOM | Assignee: | arno. <a.renevier> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, ap, a.renevier, arpitabahuguna, commit-queue, eric, esprehn+autocc, leviw, mitz, ojan.autocc, robert, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 525.x (Safari 3.2) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
URL: | http://www.bdacriacoes.com.br/googlechrome | ||||||||
Attachments: |
|
Description
Lucas Buetto
2012-11-01 04:23:49 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?
Created attachment 171889 [details]
stand-alone reduction
Yup, this is a layout bug, not a parser bug as far as I can tell. Awaiting a final patch on the other bug. *** This bug has been marked as a duplicate of bug 99442 *** This bug is not a recent regression. Is it correct that it got marked as a duplicate of recent regression? I guess the question is if it is fixed by bug 99442 or not. I don't know the answer to that. You're right in that this isn't a regression. My mistake marking it a duplicate, it's not. Created attachment 192574 [details]
patch proposal
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 on attachment 192574 [details] patch proposal Clearing flags on attachment: 192574 Committed r148027: <http://trac.webkit.org/changeset/148027> All reviewed patches have been landed. Closing bug. |