azproduction / lmd

LMD - JavaScript Module-Assembler for building better web applications :warning: Project is no longer supported :warning:
http://azproduction.ru/lmd/
MIT License
449 stars 27 forks source link

require.css on firefox #198

Open entcor opened 9 years ago

entcor commented 9 years ago

Добрый днеь !

наткнулся некорректную работу под firefox sb.require.css = function (moduleName, callback)

        function isRules(sheet) {
            if ((sheet.ownerNode || sheet.owningElement).id != id) {
                return false;
            }
            try {
                // It can be null or throw an Security error in case of cross origin stylesheets
                return !!(sheet.cssRules || sheet.rules).length;
            } catch (e) {
                // In case of access error assume that css is loaded
                return true; 
            }
        }

обращение sheet.cssRules в firefox вызывает исключение, когда css еще не загружен, хотя объект document.styleSheet[] уже создан. если return true; заменить на return false; то все работает ...

при текущей реализации в firefox callback вызывается еще до того, как стили можно использовать.

jt3k commented 9 years ago

кажется миша забил на lmd

azproduction commented 9 years ago

@alnikitich спасибо за описание проблемы. Если все так тривиально, создай, пожалуйста, Pull Request.