Don't use .getTagName(), use jQuerys .is() instead
authorDaniel Rudolf <daniel.rudolf@eifel-online.com>
Thu, 14 Feb 2013 14:43:22 +0000 (15:43 +0100)
committerDaniel Rudolf <daniel.rudolf@eifel-online.com>
Thu, 14 Feb 2013 14:43:22 +0000 (15:43 +0100)
commit404c9abecdad722acf38557c6895062cb8ce9f9a
tree4cee1ed70b8fb8a8df3905418f2d1b672a2ce87b
parent9a9c5d2cbaff147d0f657f6a3b1298de961dcdd9
Don't use .getTagName(), use jQuerys .is() instead

Your custom .getTagName() fails on jQuery selectors with multiple or no elements. When you try to .wcfFadeIn() on an selector with no elements, a JavaScript error is the result. With an selector containing multiple elements, the effect won't be changed to highlight when the first element is e.g. an div. Never ever expect a single element inside an jQuery selector, that's the first thing the jQuery doc tells you. Simply use jQuerys .is() instead - it's much more flexible.

As I can see, WCF.getEffect() is a "protected method" and only used internally. If this is not the case just say it, I'll make it downward-compatible.
wcfsetup/install/files/js/WCF.js