[Webkit-unassigned] [Bug 62098] Update layout for msub, msup, msubsup to handle script changes

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=62098





--- Comment #18 from Alex Milowski <alex at milowski.com>  2011-06-20 05:09:53 PST ---
(In reply to comment #17)
> (In reply to comment #12)
> > Meanwhile,  previousElementSibling() loops through the previous siblings just as I've done.  Adding two calls to previousElementSibling would require looping twice.  Also, that is only available on the Element class and that would require a static cast.
> > 
> > I think the code is much cleaner as it stands.
> 
> I disagree with you here.

OK.  Element position is a common criteria upon which rendering and manipulation of markup is a conditionalized.  Rather than having a previousElementSibling(), a useful method would be something like "elementPosition()".  In XPath, this would translate to count(preceding-sibling::*) or count(preceding-sibling::*)+1 depending on whether you want it one or zero based.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.