Yes, .target is working here on chromium and yes I meant Expansion I ll refactor that sorry for the wrong spelling.
On Wed, Jun 17, 2015 at 7:02 PM, Dirk Hohndel <[email protected]> wrote: > On Wed, Jun 17, 2015 at 09:59:21AM -0700, Dirk Hohndel wrote: > > > I couldn't reproduce this on Chromium but I still see the malformed > Html it > > > seems that Chromium is more tolerant to incorrect tags, I have > attached a > > > patch that fixes the tag closing, I hope this will fix the problem. > > > > Well, the HTML looks much better now, but the problem isn't fixed. > > While I am not a JS programmer by any stretch of the imagination, I tried > > debugging it a little... > > Firebug tells me that at the beginning of the toggleExpantion() function > > (umm - what's Expantion? Were you thinking of Expansion?) we derefernce > > e.toElement.localName but e.toElement is undefined... > > > > Does this help figuring out what's going on? > > Google first, hit send later... > > > http://stackoverflow.com/questions/8600174/event-toelement-in-ie8-and-firefox > > turns out that Firefox doesn't have toElement and apparently .target > should work on all browsers... > > Can you test that this works for you? > > /D > -- regards, Gehad
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
