对于现代Web前端来说,Selector是必备之物。对于标准浏览器,可用querySelector,而对于IE8标准模式以下,需要一个Selector引擎。
因为本项目会在代码里使用QWrap,所以虽然我写的部分代码并没有用QWrap,但既然怎样都需要一个Selector引擎,不如用QWrap。
结果发生一件悲剧的事情。
我使用html5-shim/shiv库来让IE正确解析新的HTML5标签。但是QWrap引擎居然不能正确选择出html5的元素。
尝试人肉选择,发现是可以的,那就是QWrap的Selector存在bug。
经过艰苦卓绝的斗争,最终发现问题出在这个函数的第2行:
function(a) { sFun.push('el.tagName=="' + a.toUpperCase() + '"'); return ''; }).replace(/([\[(].*)|#([\w\-]+)|\.([\w\-]+)/g,//id缩略写法//className缩略写法
QWrap采用了代码生成技术,即为selector拼装出对应的函数。这是一项相当 陷阱 先进的技术。
不难看出这里会产生一个tag的匹配,对于匹配“nav”所生成的代码应该是:el.tagName=="nav".toUpperCase() 。
通常浏览器对于所有HTML元素,其调用tagName返回的都是全大写字母。
【号外】为什么是大写?
Hax答:这是从HTML的祖先SGML那里继承下来的习惯。对于早期写SGML/HTML的人,要区分标签和正文内容的最简单方法恐怕就是把标签用大写。
不过,据圈子里有名的那个纹身佬说,HTML用大写是因为那时候技术落后,存储太紧张,全大写比较省存储空间……(大意如此,详情我记不清了,请八卦人士咨询Winter)
【/号外】
然而不幸的是,html5-shim以及我所知差不多所有的类似库,都会使用全小写。这是因为现代Web标准的主流是采用全小写。
【号外】为什么换小写了涅?
Hax答:因为这样比较不伤眼,也不伤手。
(每天面对满屏幕大写字母的 伤不起啊,看UPPERCASE看到神经衰弱啊, 有木有, 有木有!)
(只能用小键盘笔记本打字的 伤不起啊,打UPPERCASE打到小手指抽筋, 有木有, 有木有!)
【/号外】
而IE虽然对于它所能识别的HTML元素都是大小写不敏感的,但是对于通过 createElement神经刀产生的新元素,它其实将其视作类XML元素,也就是大小写敏感的,所以其tagName属性将返回最初设定的大小写形式。
如何fix这个问题?
一个容易想到的方式是把html5-shim里的标签列表改为大写。不过这个方式并不管用。因为tagName返回的是最初设定的值,也就是,如果你写
显然,QWrap Selector(或任何通用脚本库)不应依赖网页作者如何书写。所以这个问题必须由QW来解决。
此外,库也不应该假设tagName一定返回大写。虽然规范规定对于HTML元素tagName应该始终返回大写,但库必须考虑兼容性(即这里所提到的IE的问题)。
此外,通用脚本库也要有前瞻性,比如考虑Selector引擎用于选择XML元素。当前各种纯JS selector engine并非namespace-aware,所以本不能选择XML元素。但HTML规范已经允许直接在HTML里混合MathML、SVG。新的浏览器也都已经支持了。
比如你可以试着在FireFox里看下述代码:
你可以看到$('test').firstChild.tagName返回的是“math”而不是“MATH”。而document.querySelector('#test math')也可以正确选择到该元素。
但是QW Selector就不能选择到math元素了。
【扩展】
假如你在这个文档里插入document.createElement('math')会发生什么呢?
注意,你插入的不是一个MathML元素,那需要通过createElementNS,加上适当的namespace(http://www.w3.org/1998/Math/MathML)才可以。你插入的其实是一个名字恰好为“math”的HTML元素,该元素的tagName返回的是全大写的“MATH”。
此时,使用getElementsByTagName('math')或document.querySelectorAll('math')会返回这两个元素。而getElementsByTagName('MATH')或document.querySelectorAll('MATH')只会返回那个正好叫做“math”的HTML元素,而不会返回真正的MathML的math元素。注:FF行为如此,而目前Chrome是两者都返回的,这应该是WebKit的bug。
【/扩展】
尽管纯JS Selector引擎主要的目的是 向前兼容,但若能做到向后兼容就更好了。而QW Selector由于这个小小的大小写问题,在两方面都失败了。
好在,修正它是很容易的。
sFun.push('el.tagName=="' + a.toUpperCase() + '"');
改为
sFun.push('el.tagName.toLowerCase()=="' + a.toLowerCase() + '"');
即可。
有人可能会问,为啥全换成toLowerCase()?
之前人家木有说过吗,UPPERCASE什么的 最讨厌了! 有木有! 有木有! 有木有!
说正经的,用toLowerCase()是因为标准所规定的行为就是这样的。尽管全换成toUpperCase()似乎也没有什么不一样的。
上述代码的结果和目前Chrome的行为较为一致,即即使是非HTML namespace的元素,也按照大小写不敏感的方式比较。如果要按照FF的行为,可以改为:
'isHTMLElement(el) ? tagName.toLowerCase() == "{a.toLowerCase()}" : tagName == "{a}"'
上述isHTMLElement检测一个元素是否是HTML元素,逻辑请自行查标准确定。{a}这里用模板语法,这是为了让大家看得更明白,我也省下打许多引号和加号的力气。
好了,我已经在咆哮体上浪费太多时间了,再不交活,老板要对我咆哮了。加班去鸟。。。