search

nazarlitvin / gulp-utm2html

Add UTM tags to URL in HTML file
1 stars 2 forks source link

Игнорирование через data-атрибут #2

Closed dudeonthehorse closed 7 years ago

dudeonthehorse commented 7 years ago

Все здорово, спасибо за пакет. Забираю к себе в Kilogram.

Но в строчке if ($(links[i]).attr('data-utm') === 'nope') я бы использовал значение false или ignore – так намного семантичнее. Предлагаю добавить оба варианта с сохранением обратной совместимости.

nazarlitvin commented 7 years ago

@dudeonthehorse thanks, it makes sense, I'm going to update this ASAP.

nazarlitvin commented 7 years ago

Thanks, @drugoi for the PR #3 that provides this functionality. Now we can use one of these data-utm attributes for preventing URL change: nope, false and ignore. Updated version is already published to npm.