These are chat archives for riot/riot

15th
Jul 2015
Tsutomu Kawamura
@cognitom
Jul 15 2015 05:11
@GianlucaGuarini Sorry for my late response. It took time to prepare the environment of IE8.
IE8 doesn’t have document.head.
I’m gessing abount the polyfill of it...
Hmm.. it looks that you removed the polyfill on this commit.
riot/riot@0260943
Tsutomu Kawamura
@cognitom
Jul 15 2015 05:16
The simple solution is this, I think.
document.getElementsByTagName('head')[0] instead of document.head
Gianluca Guarini
@GianlucaGuarini
Jul 15 2015 06:31
@cognitom the thing is that with polyfills or without polyfills IE8 does never pass the riot tests
Tsutomu Kawamura
@cognitom
Jul 15 2015 06:31
Oh, really? I’ll check it again.
Tsutomu Kawamura
@cognitom
Jul 15 2015 06:49
Finaly, I found the bug.
styleNode.setAttribute('type', 'text/css’) is needed for IE8.
Tsutomu Kawamura
@cognitom
Jul 15 2015 07:04
I’m writing a PR. Hmm, it seems that we’re using document.head as a switch of Node/io.js
Ibrahim Islam
@Ibrahim-Islam
Jul 15 2015 20:36
any app sample that show Riot's router at work?