Унификация событий

[shirokoff]

Столкнулся с тем, что в nb у каждого элемента свои персональные события. Например, фокус на инпуте триггерит событие 'nb-input_focused'.

Чего мне хочется:

• Все блоки должны триггерить одинаковые события
• Названия соыбтий должны быть как в DOM с приставкой nb-. В данном случае nb-focus.

Таким образом не надо будет думать, какого вида у нас элемент. И можно будет делать что-то типа

nBlocks.forEach(function(){
    this.on('nb-focus', function(){
        alert(this.getValue());
    });
})

[pmyagkov]

Добавлю, что такой подход на один шаг приблизит нас к правильному использованию делегирования в наноостровах.

[shirokoff]

@doochik @Katochimoto @maksimr @basvasilich

[maksimr]

+1

Кстати почему не просто focus, а nb-focus?

[shirokoff]

Да, можно и просто focus наврно. Так даже лучше.

[basvasilich]

События без префикса могут пересечься с событиями в реализации
Например "open" это событие которое может прийти из вне и запустить обработчик.

Чем смущает префикс?

[basvasilich]

@vitkarpov

[shirokoff]

Ну а зачем он нужен то вообще?

[maksimr]

@basvasilich

Можно события которые имитируют или являются DOM событиями делать без префикса, а кастомные(nanoisland-овские) с префиксом nb-. Тогда сразу понятно что с префиксом это островное событие, которое к DOM никакого отношения не имеет.

[basvasilich]

Префикс чтобы не путать с DOM событиями. События в NB например сейчас не всплывают.

[arteg]

+1 к предложению унифицировать.
Насчет DOM-событий, по-моему, стандартная практика юзать свой namespace someevent.nb. События блоков можно оставить с префиксами.
А каков статус #105?

[basvasilich]

в #105 нужен rebase на master

[basvasilich]

DOM события внутри наноблоков имеют значение.
Тригерить DOM события на элементах так просто не получается.
Так что:
focus -> nb-focused
blur -> nb-blured
change -> nb-changed

[maksimr]

DOM события внутри наноблоков имеют значение.

Не совсем понятно. Какое значение?

Тригерить DOM события на элементах так просто не получается.

Почему?

[basvasilich]

В текущей реализации есть разделение на DOM и кастомные события:

DOM события навешиваются на this.node
https://github.com/yandex-ui/nanoislands/blob/master/libs/nanoblocks.js#L364-L378
Это сделано не просто так.

Все кастомные вешаются на сам nb блок, что логично.

[basvasilich]

Дальше триггер события на node вызывает обработчик внутри блока.
То есть при вызове $(this.node).trigger('blur') вызывается метод nb.block.blur() что вызывает зацикливание.

То есть либо метод называть onblur либо событие делать не DOM.
Мне кажется метод в API важнее названия события.

Такое поведение наверное можно обойти. Но там надо копать.

Нам правда так важно иметь именно focus, blur, change. Есть какие-то причины кроме эстетических когда это действительно важно?

[maksimr]

То есть при вызове $(this.node).trigger('blur') вызывается метод nb.block.blur() что вызывает зацикливание.
То есть либо метод называть onblur либо событие делать не DOM.

Есть еще вариант не вызывать trigger внутри функции blur :)
Так как эта функция обработчик события а не emitter

Нам правда так важно иметь именно focus, blur, change. Есть какие-то причины кроме эстетических когда это действительно важно?

Я думаю было важно иметь унифицированное событие :)
А с префиксом, постфиксом или без него уже вторично.

focus
nb-focus
focus.nb

а не nb-focus-input, nb-focus-button

[basvasilich]

Или я что-то не понимаю, или как ты добавишь свой обработчик на blur при:

block.blur();

Унификация уже сделана, у всех блоков nb-focused.

[maksimr]

Или я что-то не понимаю, или как ты добавишь свой обработчик на blur при block.blur()

Я бы вызвал мой обработчик там же где вызывается block.blur :) (т.е. тригерится событие blur по кторому мы вызваем block.blur)