Маленькие мелочи

Главные вкладки

Аватар пользователя batbug batbug 2 апреля 2009 в 10:29

Как известно, настроение складывается из мелочей. Когда какая-то зараза портит настроение изо дня в день, хочется удавить ее. Когда форма для ввода тэгов в друпале бесит каждый раз, когда тебе надо вбить больше одного тэга, это надо лечить.
Суть проблемы: пишем тэг, вылезает список подходящих тэгов, тыкаем в тэг, он добавляет в поле, ставим запятую, пишем следующий тэг... стоп... а какого х** я должен писать каждый раз запятую??? ну сколько можно уже??? это что нельзя добавить в скрипт??? в русской раскладке вообще запятая убого расположена! а если еще и тэги на двух языках! это же вообще бесит.

Лечение:
/misc/autocomplete.js: 158 строка

- this.input.value = this.selected.autocompleteValue;
+ this.input.value = this.selected.autocompleteValue + ',';

Аминь.

Комментарии

Аватар пользователя kyky kyky 2 апреля 2009 в 10:48

Хорош материться -- разработчиков мало волнует, где расположена запятая в русской раскладке. Это наша национальная беда.
Напиши issue Дрису, может в семерке скрипт будет с твоей поправкой.

Аватар пользователя VladSavitsky VladSavitsky 2 апреля 2009 в 13:42

neochief говорил, что для орг нужно готовить патч, причём и для 7ки, и для 6ки.
Если патч пройдёт автоматические тесты, то его могут принять.

Аватар пользователя fasdalf@fasdalf.ru fasdalf@fasdalf.ru 2 апреля 2009 в 13:55

По своему опыту, такие патчи обычно заворачивают с ответом "by design"

А так, отправь на Dru.org, будешь в резюме писать "участвовал в разарботке CMS Drupal"

Аватар пользователя sadmin sadmin 2 апреля 2009 в 15:00

то есть если мне нужен тег ул.Рябиновая д.8б, то у меня он будет разбит на два? меня ситуация с запятой не беспокоит - разделитель есть разделитель

Аватар пользователя batbug batbug 2 апреля 2009 в 17:00

"sadmin" wrote:
то есть если мне нужен тег ул.Рябиновая д.8б, то у меня он будет разбит на два?

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

Аватар пользователя PVasili PVasili 2 апреля 2009 в 18:21

Сомнительное новшество (поскольку нет такого выбора ткнул в сам дурак пиши)...
А для последнего тега нужно запятую стереть?
Если у меня по 1, 2 тега на материал то мне больше стирать, чем добавлять нужно.
Запятая нормально(по крайней мере En и Ru раскладках расположена - через 1 клавишу...)

Аватар пользователя batbug batbug 2 апреля 2009 в 20:08

"PVasili" wrote:
А для последнего тега нужно запятую стереть?

не нужно
"PVasili" wrote:
Запятая нормально(по крайней мере En и Ru раскладках расположена - через 1 клавишу...)

в русской раскладке запятая через шифт идет.

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

Аватар пользователя batbug batbug 6 апреля 2009 в 7:50

Обнаружил баг при использовании моей фичи: поле node_reference (в режиме автокомплит) не проходит валидацию, так как в конец строки node_reference добавляется запятая, а там свой формат строки. Хмм..

Аватар пользователя kyky kyky 7 апреля 2009 в 4:22

2 batbug
Стрелять так стрелять: на сабмит формы повесте функцию обрезания последней запятой. Можно сделать как в виде хука, так и jQuery-скриптом Smile

Аватар пользователя batbug batbug 7 апреля 2009 в 5:58

2 kyky
ага, можно и так, а можно и node_reference подправить, он ведь должен как-то принимать несколько значений в одном поле (это просто у меня он может содержать только одно значение)