Не работает cmd+click на ссылках

[chestozo]

Ща посмотрю, может это не сложно лечится )

[chestozo]

Можно добавить такую проверку в ns.action._process

if (e.altKey || e.ctrlKey || e.metaKey || e.shiftKey) {
    return returnValue;
}

Но нужно учесть, что через action на ссылке может висеть ns.action типа метрики и его надо успеть выполнить.

[edoroshenko]

Не работает cmd+click на ссылках

А как он должен работать? Какое поведение ожидаемо и какое по факту?

[chestozo]

cmd+click на ссылку должен открывать новую вкладку.
shift+click - новое окно (это я сам только вчера узнал ;))
Может ещё чего.

[edoroshenko]

А у каких ссылок не работает? У них есть href?

[chestozo]

Ни у каких. Есть. Можешь на вашем проекте попробовать ;)

[vitkarpov]

@chestozo имеется ввиду, что на всех ссылках в документе висит обработчик, внутри которого превентится действие браузера по умолчанию https://github.com/yandex-ui/noscript/blob/master/src/ns.action.js#L212 , если только нет target="_blank", соответственно, и такие фишки как cmd + клик или shift + клик — тоже запревентились?

[chestozo]

@vitkarpov точно!

[doochik]

Да, у нас вот такая проверка в обработчике

if (e.shiftKey || e.altKey || e.ctrlKey || e.metaKey) {
  //pass
  return; 
}

[chestozo]

йи-ха )

[doochik]

Сделаешь pr или мне?

[chestozo]

Сделаю )

[chestozo]

Мёржим?

[chestozo]

Кстати, я PR сделал прямо из issue вот так:

curl --user "chestozo" \
    --request POST \
    --data '{"issue": "225", "head": "yandex-ui:cmd.click", "base": "master"}' \
    https://api.github.com/repos/yandex-ui/noscript/pulls

Мне кажется, для мелких issue - удобно не плодить отдельный PR.
Ок?

[doochik]

👌