DrupalRU / drupal.ru

Репозиторий кода для сайта русскоязычного сообщества Drupal переехал на GitLab https://gitlab.com/drupal.ru/7
GNU General Public License v2.0
30 stars 23 forks source link

Пожалуйста, не используйте слова написанные КАПСОМ #918

Closed SemanticsPRO closed 6 years ago

SemanticsPRO commented 6 years ago

Походу, очередная претензия к работе матофильтра. Отдельные слова КАПСОМ - это НОРМАЛЬНО. ВЕСЬ ЗАГОЛОВОК КАПСОМ - НЕТ.

screenshot 102

Источник: https://drupal.ru/node/73305/edit

awd-studio commented 6 years ago

Там же написана причина. При чем тут матофильтр?

ivnish commented 6 years ago

Мы уже где-то обсуждали недавно аббревиатуры. К компромиссу не пришли

SemanticsPRO commented 6 years ago

@awd-studio наверно, это не норма. Не находишь? Или все аббревиатуры в заголовках в нижнем регистре исключительно писать? VDS/VPS vds/vps Vds/Vps

Сравните по читабельности. Общепринято в верхнем регистре все ж такое писать.

Матофильтр назван, потому что это он давал кучу приколов при редактировании/создании.

awd-studio commented 6 years ago

У нас несколько валидаторов.

Норма, не норма - хз, было определено максимальное количество в 2 символа капсом, так и работает. Можно поднять вопрос, если есть желание - составить описание, по каким критериям распознавать капс.

В скором времени планируется работа над валидаторами, если будет от чего отталкиваться, на момент реализации - внедрю.

mbaev commented 6 years ago

@awd-studio наверно, это не норма. Не находишь? Или все аббревиатуры в заголовках в нижнем регистре исключительно писать?

Задавать вопросы в агрессивном стиле ведёт к снижению продуктивности и работы команды в целом.

Изначально @awd-studio делал 3 символа, но я просил снизить до 2х. Особого сопративления моё предложение ни от кого не встретило, поэтому так и сделали. Моё мнение лучше писать в литературном стиле, где всё в нижнем регистре нежели в техническом, не смотря на то, что второе более логично. Так мы сможем исключить такие фразы, как HELP, КТО, МНЕ.

ivnish commented 6 years ago

Вот мы обсуждали уже аббревиатуры https://github.com/DrupalRu/drupal.ru/issues/861

ivnish commented 6 years ago

Так мы сможем исключить такие фразы, как HELP, КТО, МНЕ.

Это должен исправлять редактор, для этого он и есть. А сейчас мы имеем проблему, которая доставляет нам (и, главное, пользователям) головную боль

SemanticsPRO commented 6 years ago

@mbaev уже как-то игрались с литературным стилем в переводах, в итоге token был переведён как "маркер" и гордо держались за этот перевод. Ничего хорошего из этого не вышло.

Нужно, как минимум, список исключений. У нас куча известнейших терминов - VPS, VDS, PHP, CSS, HTML, CTR, SEO, IP, AJAX, CMS, DNS, FTP, HTTP. Которые мы везде видим в верхнем регистре. Принуждать писать их в нижнем - заведомо снижать читабельность.

Кроме этого, я ниразу подобной проблемы не встречал. У нас проблема с дурными заголовками, в стиле "Вопрос?!?" или "Модуль?", но никак не с капсом.

ivnish commented 6 years ago

Да, либо надо делать список исключений

mbaev commented 6 years ago

Дык потому проблемы с капсом и нет, что валидатор работает. А вот, когда он не работал: https://drupal.ru/node/135561 https://drupal.ru/node/135412 https://drupal.ru/node/135551 https://drupal.ru/node/135607 https://drupal.ru/node/135625 https://drupal.ru/node/135686 https://drupal.ru/node/129918 https://drupal.ru/node/135712 https://drupal.ru/node/135033

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

SemanticsPRO commented 6 years ago

https://drupal.ru/node/135561 Почему бы и нет, уведомление, нужно привлечь внимание

https://drupal.ru/node/135412 Ничего такого

https://drupal.ru/node/135551 Аналогично

https://drupal.ru/node/135607 Решено, это снова редакторская тема

https://drupal.ru/node/135625 Скорее к редактору/модератору, поправить мелкую описку.

https://drupal.ru/node/135686 Здесь акцент на отрицание, довольно правильный

https://drupal.ru/node/129918 URL тоже принято в верхнем регистре писать

https://drupal.ru/node/135712 Редакторская/модераторская тема

https://drupal.ru/node/135033 Название хостинга, вообще может у них торговый знак такой, в верхнем регистре.

Итого - я проблемы не вижу. Если был вал "ПАМАГИТИ ПАЛАМАЛАСЬ!", тогда да. А тут - единичные слова капсом, но при этом, мы теряем читабельность для спецов.

awd-studio commented 6 years ago

У нас куча известнейших терминов - VPS, VDS, PHP, CSS, HTML, CTR, SEO, IP, AJAX, CMS, DNS, FTP, HTTP.

+ TRUE, FALSE, NULL, ANY_OTHER_CONSTANT, etc.

+ ООО, ИП, ТОВ, ГУГЛОНЕФТЕТРЕШИНСКИЙ КОЛБАСОЛИТЕЙНЫЙ, etc.

+ CEO, NDA, FYI, etc.

+ SEO, SMM, SMS

Кароче, ннада будет тот еще список. Но желательно "на лету" чекать.

ivnish commented 6 years ago

Может его сделать на проде, чтобы редакторы-модераторы могли быстро туда сами вносить изменения? Через issue это будет та еще бюрократия

awd-studio commented 6 years ago

Во, кстати, ISSUE, GIT, BASH, PHP, EXCEL...

SemanticsPRO commented 6 years ago

@awd-studio есть ненулевой риск тупо потонуть в этом списке. Ибо Forms API, но FAPI, например.

Вообще, если на то пошло, то может иную сторону проблемы рассмотреть, а именно редакторская админка, куда ноды с подозрительными заголовками выводить? ТЗ в реалтайме, так сказать. Но не так похабно, как тикеты у модеров.

awd-studio commented 6 years ago

В пору нейроночку писать под это дело.

ivnish commented 6 years ago

Хотя я придерживаюсь мнения, что нужно вообще выключить этот валидатор. Если нашему редактору некогда править заголовки, я уже предлагал тут https://github.com/DrupalRu/drupal.ru/issues/914, что модераторский отдел готов этим заниматься

awd-studio commented 6 years ago

@SemanticsPRO все это круто, но вектор нашего развития в другую сторону. Мы нацелены на исключение человеческого фактора, а не на его усиление. Автоматизируем.

А админку - я писал. За что и был "награжден" ее использовать. Нарекания прошу более конструктивно, или ауторерайтить.

SemanticsPRO commented 6 years ago

и до админки доберёмся, я с этой только недавно разобрался

SemanticsPRO commented 6 years ago

@awd-studio это всё импосибль. И что-то говорит мне о том, что мы 0.5 человеко-часа редактора в месяц хотим сменить на 5 человеко-часов разработчика в месяц. Будут возникать проблемы, будут возникать вопросы, будут возникать возражения и негативный UX.

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

ivnish commented 6 years ago

@mbaev сам неоднократно говорил, что не нужно ущемлять пользователей. А невозможность написать в заголовке "VDS" или "PHP" - это что ни на есть самое настоящее ущемление. Причем не только пользователя, но и модератора/редактора

awd-studio commented 6 years ago

Пассбл-пассбл... Все пассбл. С таким настроем у нас бы все "вручную" работало бы.

@itcrowd72 Макс был за то что бы все было более юзерфрендли, а не против (на сколько я помню). Например, вместо упрека в виде сообщения об ошибке после отправки формы, можно подсказывать пользователю "налету", и даже предлагать исправить (или отправить запрос на добавления в исключения, если по мотивам этого ТОПа).

mbaev commented 6 years ago

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

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

Есть другой вариант, написать граббер, который будет:

  1. собирать все слова, написанные капсом в заголовках
  2. складывать в словарь
  3. выводить в админский UI
  4. уполномоченный человек/люди отмечает(ют) "запрещённые" для капса слова

Со временем так наберётся критическая масса и система будет работать надёжно.

SemanticsPRO commented 6 years ago

@mbaev опять же, не знаю, регулярка будет монстрообразная. Современные процы её наверняка вытянут, даже если там будет 1000 элементов.

Уж лучше тогда начать статистику собирать.

adubovskoy commented 6 years ago

Мы тут не решим "надо или не надо". Нужно больше людей, голосов и отзывов.

я за то чтобы убрать плохо работающий функционал. Как ты выразился в соседнем issue - "но не будем закрывать вопрос вообще, а просто отложим пока не сделаем успешную техническую реализацию". Кхе-кхе.

А как только кто-нибудь пришлет регулярку которая пройдет тесты - сразу включим :)

ivnish commented 6 years ago

я за то чтобы убрать плохо работающий функционал.

Полностью поддерживаю. Пока не будет нормальной реализации, эту нужно выключить

А новая будет тогда, когда найдется желающий её написать (это может быть очень нескоро)

mbaev commented 6 years ago

Или наоборот, выключим как только кто-то напишет что-то лучше. А то если двигаться по этой стратегии, то придётся выключить половину сайта: контактную форму, редактор текста, трекер.

Предлагаю всё таки продолжать давить и может быть кто-то из здесь присутствующих сломается! image

adubovskoy commented 6 years ago

Мне пофигу, я пока писать никакой контент с этой штукой не хочу) и так мало мотивации писать в паблик, а не в личный блог, так еще и этот фильтр - нет, спасибо)

mbaev commented 6 years ago

Говоришь как обычный пользователь)

bsyomov commented 6 years ago

Не будет никогда нормальной реализации этого фильтра - белый список не реален, чёрный список не реален. Оба из-за огромного количества вариантов. Нейронка не имеет смысла. =)

Просто надо отключить этот фильтр. Он вреден. Если будет клинический случай, в виде всего капсом, то надо просто "давать по шапке" автору...

mbaev commented 6 years ago

Ладно, @awd-studio их больше. Мы проиграли =)

awd-studio commented 6 years ago

Та не вопрос - сымай галку :)

awd-studio commented 6 years ago

Сделать PRом? Или переделать на проверку полной фразы на КАПС?

mbaev commented 6 years ago

@awd-studio а можно сделать админку для того, чтобы можно было забить слова, которые нельзя писать капсом? Просто textarea, где вводятся слова через запятую. м?

awd-studio commented 6 years ago

Канеш можно. Это у меня даже в ТУДУхе есть было. Могу даже сделать еще целую галку из разряда "Чекать всю фразу". Сделать?

mbaev commented 6 years ago

Да, было бы супер. Начнём с пустого поля, а потом будем добавлять слова HELP, ПАМАГИТИИ и т.д. Только нужен ещё пермишен отдельный на админку, чтобы модераторы могли её трогать.

awd-studio commented 6 years ago

Пермишн там должен быть, если нет - добавлю. (Так эта... а отключать не будем? или добавить убрать проверку заголовков в блогах?)

ivnish commented 6 years ago

Я так понял @mbaev просит всё переделать на работу с черным списком. Типа всё разрешено, что не в этом списке

mbaev commented 6 years ago

@itcrowd72 да, верно.

awd-studio commented 6 years ago

Это я понял, я про конкретно блоговый тайтл уточнить хочу. Нужно ли отключать его проверку этим валидатором.

mbaev commented 6 years ago

Не, пусть останется. Просто после переделки "стоп-словаря" не будет и валидатор будет пропускать все слова.

awd-studio commented 6 years ago

Принял.