:)

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

спасибо, я унижен, всем пока

А где removeEventListener?

Лёша

Мне и так уже плохо. Зачем ты это пишешь?

Я все ж, спробую написати по суті питання хоча б 😅.

Коротше, залишати рядки — нормально, якщо вони такі дескриптивні як тут.

Якщо ж дотримуватися DRY, то варто починати з найбільш зовнішнього куска. Наприклад, винести все в хелпер addClickListener, і тоді стане очевидно що винесення константи із такого ван-лайнера - справа смаку.

Можливо, лише для тестів було б корисно таке виносити, щоб не повторювати там рядки.

думаю, скорее вопрос не про DRY, а про то что в строках можно опечататься и ужас-ужас может быть баг, поэтому лучше импортить константы, они же статически анализируются

Ну хелпер теж статично аналізується, імовірність зробити помилку буде та ж сама

Не то что бы проблема ) решение не очень чистое как по мне, редактировать бандл после билда, собственно искал альтернативы

можна тайпскриптом рядочки аналізувати

ти click хочеш у константу винести? імхо в даному випадку не треба, це комон ноледж

Мене взагалі трішки бісить, що ми маємо думати над такими рішеннями 😁

Это какая-то надуманная проблема и тупое решение без какой либо пользы. Не помню хотя бы раз кто-то страдал от проблем в опечатке ивента

Хелперы поверх aEL которые не делают ничего кроме как определяют ивент — ещё бессмысленнее

у меня бьіло пару раз, но не ивента, а чего-то другого

например вместо click написали press или push когда речь о button

А потом не заработало и долго думал почему? При том, что вынесение этой строки в констану ни как не решит вопрос )

Привет. У меня очередной дурацкий вопрос:
Есть рейтинг.
Я решил прикрутить к нему https://schema.org/AggregateRating
Ибо нехрен тут без схемы.
Так как страница — адская солянка из компонентов, то я бы хотел добавить его ввиде microdata. Ну и концептуально это, по моему, лучший подход.
Но рейтинг как таковой показан просто количеством звездочек.
Там нет интеджера с количеством звёзд 🙁
Добавить <span itemprop="ratingValue">4</span> и спрятать — чувствую что это порочно.
В качестве аттрибута неясно как. Хотя думаю могло бы быть неплохо. Сейчас обертка так выглядит:

<StyledWrapper
          itemprop="aggregateRating"
          itemscope
          itemtype="https://schema.org/AggregateRating"
          aria-label={i18n._(t`Rating: ${score} out of 5`)}
          {...rest}
        >

Вменяемая альтернатива — добавлять как JSON-LD, но либо я тогда дублирую данные, либо добавляютуда только рейтинг, но я не уверен как вместе будет работать JSON-LD+microdata. Смерджатся или будут два неполных набора данных?

Что кстати лучше — microdata или RDFa?

Cудя по всему RDFa стоит использовать…

+1 к RDFa. Microdata вроде бы давно мертвый стандарт, нет?

Начет как расположить. Я думаю, а если не просто span, а использовать тег template. Хз, поймает ли его робот, но скрин ридер и прочие таки штуки template точно будут игнорить.