Pull to refresh

Comments 58

Спасибо, я не знал о ней.
Интересна идея вытаскивать из html конкретный блок.
Жаль только, что вложенность тегов не повторяется.
Я автор этого грант-плагина. Если есть вопросы, буду рад :)
Жаль только, что вложенность тегов не повторяется.
Обычно, когда верстаю, элементы складываю на один уровень вложенности. Если нужен второй, создаю новый блок. Делал плагин исходя из такой иерархии.
Да, я так и понял. Это чисто БЭМ-подход.
Мне хотелось сделать инструмент, который не будет полностью завязан на методологии, для большей свободы.
Добавьте возможность использования регелурных выражений в ignore, т.к. в большинстве случаев, просто используется префикс:

<div class="foo__bar_baz js-foo__bar_baz" />


И еще, сделайте опции в виде хеша, а не методов.
Добавил issue про ignore.
Сначала я тоже хотел сделать опции в виде хеша, но потом передумал в пользу отдельных методов, поскольку это выразительнее и гибче.
Опции всегда должны описываться в декларативном стиле.
Сначала я тоже хотел сделать опции в виде хеша, но потом передумал в пользу отдельных методов, поскольку это выразительнее и гибче

Насчет гибкости не понял. За счет чего она должна достигатья?
Как-правило, настройки все хранят в отдельном файле, в вашем же случае без «магии» не обойтись.

Представьте что у вас есть проект, который состоит из множества подпроектов.
Намного проще определить конфиг в котором описываются базовые опции, в при использовании их можно переопределить.

# config.yaml

brace:  ['newline']
ignore: ['js-.*']
flat:      true
indent: 'tabs'
inner:   true
line:     true


// project1.js

var options = Object.mixin({
  tag: 'true',
  ignore: ['foo']
}, options);

new Autoclasscss(options);

// project2.js

var options = Object.mixin({
  ignore: ['bar', 'js-*']
}, options);

new Autoclasscss(options);


Например, ignore добавляет игнорируемые классы, а не полностью переопределяет заданный ранее список.

Можно сделать обёртку, которая будет возвращать экземпляр со стандартными настройками.
return (function() {
    return new Autoclasscss().tag(true).flat(false);
})();

И на проектах использовать её, переопределяя необходимые опции.
require('autoclasscss').tag(false).set('<html>').get();
Все-таки опции должны оставаться оставаться опциями, а не методами.

Представтьте что, кто-то захочет добавить ваш плагин в грант.
Зачем вы используете window?

var __global__ = typeof global !== 'undefined' ? global : function (object) {
    return object;
}(this);
Тогда уж:

Function ('return this')();

Но зачем?
Чтобы гарантированно получить глобальный контекст + минимум кода.
eval в первую очередь зло.
А минимум кода должено должно достигаться за счет сжатия
1. Не будьте так категоричны
2. Function(/*body*/) — это форма eval
3. Сжимальщику иногда нужно помогать.
1. Не будьте так категоричны

Просто так не нужно писать, тем более для таких простых вещей как получение ссылки на глобальный объект.

2. Function(/*body*/) — это форма eval

Именно по этому я ее и привел, т.к. это почти единственная форма динамического исполнения кода, которая допустима в строгом режиме.

3. Сжимальщику иногда нужно помогать.

Вот что должно помогать developers.google.com/closure/compiler/docs/js-for-compiler
Почему нельзя просто?
(function(global) {

})(this);
Потому что так не получится, если используется AMD.
Да, но у меня не используется AMD.
А если надо применить этот скрипт в AMD, то его достаточно вызвать в нужном контексте.
Ага, понимание того как и где это может испольваться приходит не сразу!
Конечно можно, только так:

(function(global) {
  //..
}).call(this);


Все зависит от проекта, кому как удобно.
Да и замыкания не бесплатны, особенно в больших проектах, т.к. они меннее эффективно развоваричаются.
Только то, что вы написали — неправильно. global у вас будет undefined.
Я хотел показать что в таком случае this будет ссылаться на глобальный объект;
Хотя конечно, лучше оставить global, чем передавать два объекта
'use strict';

(function () {
  alert(this);
}).call(this);


Я не понимаю, что вы мне пытаетесь доказать?
!function () { 'use strict';

(function () {
  alert(this);
}).call(this); }()
Вы первый начали это троллинг.

Просто я действительно не понимаю зачем использовать eval, если можно использовать конструктор Function (пусть даже это тоже eval)?
Хотя я конечно согласен, что для библиотеки, которая будет использоваться «не пойми как», без «eval» для получения ссылки на глобальный объект не обойтись.
Просто так не нужно писать, тем более для таких простых вещей как получение ссылки на глобальный объект.
Расскажите этим ребятам github.com/knockout/knockout/blob/master/build/fragments/extern-pre.js#L4

единственная форма динамического исполнения кода, которая допустима в строгом режиме.
Вот врать-то зачем? eval прекрасно работает в strict mode.
Вот врать-то зачем? eval прекрасно работает в strict mode.

The eval code cannot instantiate variable or function bindings in the variable environment of the calling context that invoked the eval if either the code of the calling context or the eval code is strict code. Instead such bindings are instantiated in a new VariableEnvironment that is only accessible to the eval code.

Такая конструкция?
(0,eval)('this')

А вот такое заработает?
(function eval () { 
  'use strict'; 
    alert(this);
})();
1. В strict mode eval сделали безопаснее, об этом и речь в приведенной цитате. Т.е. все что происходит внутри eval — остается внутри eval.

2. Причем тут вообще второй пример? Он не будет работать, потому что в strict mode нельзя переопределять eval и arguments.

Вот, вам чтиво на вечер sriku.org/blog/2012/08/28/on-eval-being-evil/
Не нужно мне кидать всякие левые сслылки, в спецификации все и так подробно написано.

Вам нужно провериться у психоаналитика, у вас походу эгоцентризм.
Ну или менее экзотическое:

(function() {
    'use strict';

    eval('var x = 1;');

    alert(x);  // Uncaught ReferenceError: x is not defined 
}());
Все, правильно — eval стал безопаснее, и не засоряет вызывающий контекст лишними переменными.

Хотите объявлять переменные в родительском контексте, делайте так: this.x = 1; никогда не делайте так, и так, как в примере выше.

(function() {
    'use strict';

    eval('this.x = 1;');

    alert(x); // 1
}.call(this)); //нужно, т.к. иначе в strict mode this будет undefined.
(function () {
    'use strict';
    alert(Function('return this')());
})();

отсюда.
по-моему самый «человечный» вариант, как на мой взгляд, нежели запятые и eval:
Functions that are created with the Function constructor are strict only if they start with a Use Strict Directive, they don't «inherit» the strictness of the current context as Function Declarations or Function Expressions do

Я имел ввиду не полностью аналогично, а примерно так:

"Function, в отношении объявления переменных всегда работает аналогично eval'у в strict mode."
За одним лишь исключением — код подаваемый в конструктор Function, в отличии от eval может быть оптимизирован (в V8 точно).
Что можно оптимизировать в 'this' или 'return this'?
Потому что рассчитываю на использование только в браузере.
Как вы думаете, нужен ли CLI для этой штуки?
Можно настроить свой редактор кода, проксю, прикрутить мониторинг на изменение файла, но никто не будет открывать браузер и копировать туда разметку чтобы получить CSS.
Если настрить вывод в определенный файл, в котором уже есть стили, то они перезапишутся.
Вы же сами настраиваете вывод, сделайте так, чтобы он добавлялся в конец :)
А на сколько сложно будет сделать чтобы правила добавлялись в нужном семантическом порядке?

Речь идёт о сортировке css-свойств?
Для этого есть классная штука, csscomb.
Представьте что у меня есть такая разметка:

<header class="page">
    <span class="page__element">
</header>


CSS-код, который получился:

.page {
    
}
    .page__element {
        
    }


Затем я добавтл стили:

.page {
      width: 100px;
}
    .page__element {
          color: red;
    }



И через какое-то время, я решил добавить еще один модификатор в разметку:

<header class="page">
    <span class="page__element page__element_gren">
</header>


И ожидаю что в стилях будет следующее:

.page {
       width: 100px;   
}
    .page__element {
        color: red;
    }
        .page__element_gren {

        }


А на деле:

.page {
    
}
    .page__element {
        
    }
    .page__element_gren {
        
    }


Понял идею. Это конечно круто, но тяжело будет сделать.
Если что, могу рассмотреть пулреквест :)
возможно кому-то пригодится github.com/generalov/bem-reverse

split-css.js — раскладывает блочный css на отдельные файлы для элементов и модификаторов внутри одного уровня переопределения. полезно для быстрого прототипирования.
reverse-html.js — строит bemjson по html с классами в bem-стиле
reverse-css.js — распиливает скомпилированную css, с учетом уровней переопределения.

писалось, для о-bem-ливания верстки прототипов и изучения логики bem-tools на примере реальных яндексных сайтов. кстати, благодаря БЭМ код у них получается хорошо отделимым. =)
неплохо бы это все в ридми добавить :)
Спрашивал у Хабр жителей такое решение.
Вот что советовали:
javascript:var g=[];for(var i in document.all) if(document.all.hasOwnProperty(i) && document.all[i].id) g.push('#'+document.all[i].id); alert(g.join("\n")); автор CAH4A
Sign up to leave a comment.

Articles