Обсуждение MediaWiki:Gadget-ListingEditor.js

Перенос новой версии Listing Editor из английского раздела

Я ещё раз посмотрел, как работает новая английская версия, и не обнаружил никаких особых отличий от старой, кроме появления Edit summary и minor change. Или я что-то важное упустил? Если нет, то намного эффективнее в плане временных затрат просто добавить новую функциональность к уже существующему скрипту, чем зачем-то портировать новую английскую версию. На что ушло бы несколько дней, а результатом оказался бы код, просто более приближенный к англ. версии (ОК, плюс немного лучше написанный).

Предлагаю создать список пожеланий, что хотелось бы добавить/поменять в Listing editor. Что приходит в голову:

• По возможности автоматически подстраивать ширину эдитора под экран. Не знаю, насколько это возможно, но было бы неплохо.
• Из символов валют оставить только евро и рубль.
• Рядом с названием добавить возможность вставлять «».
• Добавить Edit summary и minor change.

Что-нибудь ещё?--Kiaora (обсуждение) 12:18, 13 апреля 2016 (MSK)

Согласен по всем пунктам. Новых мыслей нет, но я listing editor'ом практически не пользуюсь. --Alexander (обсуждение) 15:59, 13 апреля 2016 (MSK)

• В английской версии добавили возможность вводить wikidata id и по нему получать координаты,картинку,ссылку на википедию и, может быть, что-то ещё. Мне кажется, это полезной функцией. Вопрос в следующем: у нас нет такого поля как ссылка на википедию, предполагается ли её добавление?

Помнится, когда-то был разговор, что эти ссылки будут автоматически запрашиваться с wikidata и отображаться при рендеринге страницы, если я ничего не путаю. Эта идея всё ещё жива? --Kiaora (обсуждение) 17:36, 13 августа 2016 (MSK)

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

В остальном, мне пока не до конца ясна сама логика этого подхода — брать что-то с Викиданных и автоматически копировать сюда (см. также свежее обсуждение в английской Пивной). Казалось бы, уж ссылку на Википедию можно было считывать шаблоном напрямую, но никто даже не попытался этого сделать. Мне кажется, что в конечном итоге это соответствует логике английского раздела "держать всё у себя", из-за чего они, например, жёстко прописывают имена баннеров и не получают новые (лучшие) баннеры, когда мы их делаем и на Викиданных меняем. Это, по-моему, тупиковый путь развития. --Alexander (обсуждение) 18:12, 13 августа 2016 (MSK)

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

А по теме листинг эдитора - нужна ли новая кнопка, которая бы считывала хотя бы координаты и картинку из викидаты? --Kiaora (обсуждение) 06:21, 17 августа 2016 (MSK)

Мне не до конца понятно, есть ли там проблемы с большим числом запросов (User:Matroc якобы что-то тестировал, но у меня есть сомнения в его адекватности), а если они есть, то вряд ли Викиданным об этом вообще сообщили. К сожалению, нет ни одного человека, систематически занимающегося интеграцией с Викиданными, поэтому не совсем ясно, что вообще происходит и какой общий план развития (скорее всего, его просто нет).

Cчитывание из Викиданных — да, можно сделать. --Alexander (обсуждение) 14:05, 17 августа 2016 (MSK)

@Atsirlin: Могу попытаться сделать. Ещё нужно? 217.117.125.83 13:37, 29 июня 2022 (MSK)

• Новая версия готова, но там точно есть ошибки, было бы неплохо их отладить. Посмотрел, как это происходило в прошлый раз: Alexander переименовывал MediaWiki:Gadget-ListingEditor.js в ListingEditor.js (как это следует из истории изменений MediaWiki:Gadget-ListingEditor.js) и, наверное, где-то в списке подключаемых плагинов тоже заменял имя плагина на новое. А дальше править ListingEditor.js у меня уже есть возможность. --Kiaora (обсуждение) 11:05, 3 ноября 2016 (MSK)

Спасибо! Я уже тоже не помню, но, мне кажется, надо ещё раз создать страницу ListingEditor.js, подцепить её себе лично через common.js, отключив при этом в настройках старый Listing Editor. Потом отладить, после чего я перенесу новый вариант в пространство MediaWiki. --Alexander (обсуждение) 11:28, 3 ноября 2016 (MSK)

Alexander, перенесите, пожалуйста, в MediaWiki. Возможно, при вызове ListingEditor'а будет вылетать ошибка "autocomplete is not a function", но, чтобы мне с этим разобраться, нужно в любом случае перенести скрипт в MediaWiki пространство. Плюс потом ещё нужно будет подправить CSS правила. --Kiaora (обсуждение) 16:42, 4 ноября 2016 (MSK)

Андрей, спасибо! Скопировал скрипт в MediaWiki пространство. Изменений пока не вижу, но, может быть, надо подождать. --Alexander (обсуждение) 17:05, 4 ноября 2016 (MSK)

Теперь вижу: окно просто не открывается. --Alexander (обсуждение) 17:07, 4 ноября 2016 (MSK)

Да не за что, уж слишком долго я откладывал написание этого скрипта :)

И я предполагал, что эта проблема может появиться (возникает джаваскиптовая ошибка "autocomplete is not a function"). Проблема в том, что движок не может найти функцию autocomplete(), входящую в состав jquery-ui библиотеки. В английском разделе она загружается автоматически при открытии страницы вот в этом запросе. Не знаете, как её включить/подключить и в русском разделе? Если не найдёте, можно будет спросить на англ WV у авторов ListingEditor'а и на время закомментировать в скрипте строчки c 598-й по 621-ю и ещё 886-ю строчку, это чтобы эдитор снова стал работать, только без этой функциональности. --Kiaora (обсуждение) 19:43, 4 ноября 2016 (MSK)

Я нашёл у Ryan'а в обсуждении, что надо добавить dependencies в MediaWiki:Gadgets-definition, но это не помогает. Буду спрашивать дальше. --Alexander (обсуждение) 20:14, 4 ноября 2016 (MSK)

Да, почему-то не загружается этот модуль. Там в файле, на который я ссылался выше, в первой строчке идёт mw.loader.implement("jquery.ui.autocomplete@0y462qu",function($,jQuery,require,module){( ... и дальше идёт сама функция. В русском же разделе такой строчки нет в подгружаемых Javascript файлах.

Может быть, в MediaWiki:Gadgets-definition прописываются только зависимости, но при этом эти библиотеки автоматически не подгружаются. Скорее всего так.

Спасибо, что взялись за дело :) --Kiaora (обсуждение) 20:35, 4 ноября 2016 (MSK)

Андрей, а нужно ли что-то в css добавлять? Сейчас поля выстраиваются в одну колонку, что некрасиво и неудобно. --Alexander (обсуждение) 20:27, 4 ноября 2016 (MSK)

Да, нужно будет некоторые правила из англ раздела перенести, я займусь этим. Только может быть завтра закончу, если там их много. --Kiaora (обсуждение) 20:35, 4 ноября 2016 (MSK)

В нашем MediaWiki:Gadget-ListingEditor.css удалите все строчки до /* **RUS** added the following class */. И на место удалённых строк вставьте все правила из англ Gadget-ListingEditor.css. После этого останется только немного подправить. --Kiaora (обсуждение) 20:51, 4 ноября 2016 (MSK)

Сейчас всё работает как надо. Надо было дольше ждать или несколько раз чистить кэш. Андрей, ещё раз спасибо! --Alexander (обсуждение) 23:12, 4 ноября 2016 (MSK)

А, понятно. Вам тоже спасибо :) И можете ещё разок перенести с моего пользовательского пространства в MediaWiki ListingEditor.js и ListingEditor.css файлы? Исправил пару недочётов. --Kiaora (обсуждение) 16:46, 5 ноября 2016 (MSK)

Перенёс. --Alexander (обсуждение) 23:31, 5 ноября 2016 (MSK)

И убрал строку про underline, потому что она приводит к подчёркиванию вообще всех ссылок на странице. Нужно, видимо, как-то иначе это написать. --Alexander (обсуждение) 23:39, 5 ноября 2016 (MSK)

Подправил. --Kiaora (обсуждение) 17:47, 6 ноября 2016 (MSK)

Почему-то у меня не добавляются идентификаторы Викиданных, см. историю статьи Берхтесгаден.--Ymblanter (обсуждение) 11:24, 6 ноября 2016 (MSK)

Как это работает у меня. В случае, если я ввёл что-то в поле Викиданных и после этого выбрал значение из выпадающего списка (что привело к добавлению двух новых ссылок справа "Q404776" | "удалить" и "Импортировать информацию со страницы Викиданных" снизу), то ListingEditor всё сохранит. В случае же, если я ввёл что-то в поле Викиданные, но ничего не выбрал из выпадающего списка - то ничего не будет сохраняться. Так ли это работает у Вас? --Kiaora (обсуждение) 17:47, 6 ноября 2016 (MSK)

Да, кажется, так работает, спасибо.--Ymblanter (обсуждение) 17:53, 6 ноября 2016 (MSK)

Не за что--Kiaora (обсуждение) 18:49, 6 ноября 2016 (MSK)

Т.е. это фича на тот случай, если кто-то ввёл в качестве ID Викиданных несуществующий номер? Не совсем интуитивно, но, пожалуй, полезно. --Alexander (обсуждение) 17:59, 6 ноября 2016 (MSK)

Сложно сказать наверняка, какая именно идея была у разработчиков. Но в том числе так было сделано, чтобы разделить две сущности - имя в Викиданных (которое сейчас отображается в поле Викиданных) и номер Викиданных (который отображается справа в виде ссылки и который как раз будет сохранён). Если открыть код страницы, за эти два значения отвечают два разных input c id="input-wdid-label" и id="input-wdid-value", последний как раз и сохраняется. Но теперешнюю имплементацию можно и поменять при необходимости. Например, всегда сохранять значение номера Викиданных из input-wdid-value, если оно присутствует, а если нет - то брать значение из input-wdid-label, но ,естественно, в input-wdid-label может быть какая угодно абракадабра. --Kiaora (обсуждение) 18:49, 6 ноября 2016 (MSK)

Я бы оставил как есть, если ни у кого возражений не будет. --Alexander (обсуждение) 18:54, 6 ноября 2016 (MSK)

@Андрей. +вопрос про подчёркивания выше. --Alexander (обсуждение) 11:32, 6 ноября 2016 (MSK)

Андрей, не добавите ли Вы при случае в окно редактирования длинное тире? Кавычки там уже есть, а вот тире очень не хватает. Спасибо! --Alexander (обсуждение) 17:49, 11 ноября 2016 (MSK)

Сделано. --Kiaora (обсуждение) 20:10, 11 ноября 2016 (MSK)

Нестандартный тип листинга

Можно ли сделать, чтобы в таких случаях значение параметра type менялось не на null, а на other? GMM (обсуждение) 15:13, 18 августа 2016 (MSK)

Чтобы это исправить, мне нужно понять, как произошло, что type=listing поменялся на type=null. Я пробую это повторить на тестовой странице и не получается. Листинг эдитор в том случае вообще неправильно работает, но type=listing на type=null не меняет. Или вот ещё, но тоже на null не меняется. Нужен пример, когда бы эта ошибка повторялась.--Kiaora (обсуждение) 16:00, 19 августа 2016 (MSK)

У меня воспроизвелось: открыл исходный листинг, добавил координаты, нажал «Сохранить». Как еще потестировать?.. GMM (обсуждение) 16:57, 19 августа 2016 (MSK)

А у меня всё работает нормально :) , при чём и в разных браузерах тоже. Могу предположить, что мы используем разные браузеры или их сильно отличные версии (пользуетесь ли Вы IE/Edge?), либо набор включённых плагинов (в т.ч. плагинов Викигида и плагинов в браузере) разный и это как-то влияет на работу листинг эдитора. Странно, но могу поверить. --Kiaora (обсуждение) 20:05, 21 августа 2016 (MSK)

Я думал, javascript везде интерпретируется одинаково :) Но действительно, в IE 11 все работает правильно, а в последнем Chrome и Palemoon форма выглядит вот так и при нажатии «Сохранить» записывается null. Плагины в браузере — это флеш и java? У меня они отключены. Ладно, давайте отложим до лучших времен: проблема не стоит того, чтобы писать разный код под разные браузеры. GMM (обсуждение) 14:49, 22 августа 2016 (MSK)

* при добавлении листинга

Новые элементы добавляются с «*» перед ними, что, вроде бы, противоречит принятому оформлению, да и раньше так не было. --Bok (обсуждение) 01:35, 22 декабря 2016 (MSK)

Прошу прощения, почти забыл об этом. Подтверждаю, что listing'и добавляются со звёздочкой. Андрей, что скажете? --Alexander (обсуждение) 11:57, 25 декабря 2016 (MSK)

Я посмотрел на код, и, насколько я понял, звёздочки добавляются в updateSectionTextWithAddedListing. --Bok (обсуждение) 01:22, 29 декабря 2016 (MSK)

Убрал в этом месте звёздочки. Кажется, всё получилось. --Alexander (обсуждение) 06:19, 29 декабря 2016 (MSK)

Listing Editor not working

Я не вижу "карандашик", и тем самым не могу редактировать на этой странице листинги индивидуально, через скрипт. Это только у меня такая проблема?--Ymblanter (обсуждение) 09:57, 11 мая 2017 (MSK)

Нет, кажется, они везде пропали, не только тут.--Ymblanter (обсуждение) 10:06, 11 мая 2017 (MSK)

Я тоже не вижу карандашики. В английском разделе всё работает, но там текстовая ссылка. Андрей, что скажете? --Alexander (обсуждение) 10:33, 11 мая 2017 (MSK)

Если разлогиниться, то всё работает нормально. И после листинга добавляется span class="vcard-edit-button noprint":

<span class="listing-metadata"> <span class="listing-metadata-items"><span class="listing-lastedit" title="Последнее обновление">май 2017</span> <span class="vcard-edit-button noprint"><a href="javascript:" class="icon-pencil" title="Редактировать объект">редактировать</a></span></span></span>.

Если же быть залогиненным, то этот span не добавляется. Впоследствии листинг эдитор не находит его и не создаёт ссылки "Редактировать объект".

<span class="listing-metadata"> <span class="listing-metadata-items"><span class="listing-lastedit" title="Последнее обновление">май 2017</span></span></span>

Это нужно открыть не листинг эдитор, а какой-то другой файл, который создаёт всю html страницу, и посмотреть, что там происходит. Alexander, не помните, какой именно это файл делает?--Kiaora (обсуждение) 14:09, 11 мая 2017 (MSK)

Андрей, нет, эта штука, span class="vcard-edit-button noprint", добавляется через MediaWiki:Gadget-ListingEditor.js. Я подозреваю, что начала ложно срабатывать какая-то проверка, которая отключает добавление listing'ов, поскольку ссылка "добавить объект" тоже исчезла.

Кстати, я даже в разлогиненном состоянии карандашиков не вижу. --Alexander (обсуждение) 15:01, 11 мая 2017 (MSK)

А, да, Вы правы. Там, скорее всего, вот эта функция стала выдавать false в залогиненном состоянии.

/** * Return false if the current page should not enable the listing editor. * Examples where the listing editor should not be enabled include talk * pages, edit pages, history pages, etc. */ var listingEditorAllowedForCurrentPage = function() { var namespace = mw.config.get( 'wgNamespaceNumber' ); if (namespace !== 0 && namespace !== 2 && namespace !== 4) { return false; } if ( mw.config.get('wgAction') != 'view' || $('#mw-revision-info').length || mw.config.get('wgCurRevisionId') != mw.config.get('wgRevisionId') || $('#ca-viewsource').length ) { return false; } return true; }; --Kiaora (обсуждение) 15:33, 11 мая 2017 (MSK)

Причём в английском разделе вся инициализация листинг эдитора, и в том числе эта функция, такие же как и в русской. И там всё работает. --Kiaora (обсуждение) 15:50, 11 мая 2017 (MSK)

Попробовал закомментировать эту часть, но не помогло. Или я не то закомментировал? --Alexander (обсуждение) 16:01, 11 мая 2017 (MSK)

Андрей, я думаю, что-то поехало при последнем обновлении движка. Посмотрите вот тут. Я вижу там имеющую к нам некоторое отношение строчку "git #9f3bec98 - jquery.ui: Avoid deprecated jQuery.expr[":"]. Может, она что-то подскажет? --Alexander (обсуждение) 16:16, 11 мая 2017 (MSK)

Вы всё правильно закомментировали. Тогда предположу, что файл Gadget-ListingEditor.js вообще не подгружается. Попробуйте добавить alert("Test"); перед вызовом wrapContent(); , чтобы проверить это предположение. --Kiaora (обсуждение) 16:52, 11 мая 2017 (MSK)

Javascript на страницах ещё выдаёт warning Gadget "ListingEditor" styles loaded twice. Migrate to type=general. See <https://www.mediawiki.org/wiki/RL/MGU#Gadget_type>. Заодно, кстати, слетели и баннеры на заглавной, они все сразу отображаются. --Bok (обсуждение) 16:20, 11 мая 2017 (MSK)

Да, с этим тоже нужно будет потом разобраться. Но эта проблема вряд ли мешает листинг эдитору запускаться в первую очередь.--Kiaora (обсуждение) 16:52, 11 мая 2017 (MSK)

Евгений, спасибо. Всё дело было в этом type=general. Сейчас у меня и карандашики появились, и карусель на заглавной заработала. --Alexander (обсуждение) 17:13, 11 мая 2017 (MSK)

Неожиданно. --Kiaora (обсуждение) 23:17, 11 мая 2017 (MSK)

@User:Whatamidoing (WMF), we need your help. Listing Editor stopped working, presumably after the Mediawiki update last night, but we do not understand why. Do you know who could help us with this issue? --Alexander (обсуждение) 16:16, 11 мая 2017 (MSK)

I hope that we solved the problem ourselves. Sorry for disturbing you=) --Alexander (обсуждение) 17:29, 11 мая 2017 (MSK)

Удаление редко используемых полей

Можно было бы убрать поля Скайп и E-mail из эдитора, чтобы не захламлять рабочее пространство, к тому же они практически никогда не заполняются. При этом эти данные можно будет продолжать править, используя обычный редактор страниц.

Вместо них добавить Фейсбук, который может быть актуальным в заграничных объектах. --Kiaora (обсуждение) 14:48, 11 июля 2017 (MSK)

Удалить скайп и e-mail я не против, а вот facebook вроде бы уже есть. --Alexander (обсуждение) 14:51, 11 июля 2017 (MSK)

Вы имеете ввиду удалить скайп и e-mail совсем из всех листингов или только не отображать в листинг эдиторе? Просто поля фейсбук в листинг эдиторе нет. --Kiaora (обсуждение) 14:57, 11 июля 2017 (MSK)

Я имею в виду удалить из listing editor'а. А поле Facebook у меня отображается, только сегодня им пользовался. --Alexander (обсуждение) 15:05, 11 июля 2017 (MSK)

Забавно, у меня в Firefox и Safari поле facebook вырезается AdBlock Plus аддоном. Чтобы исправить, нужно будет, наверное, поменять id у этого элемента на что-то менее провокационное.

А с полями скайп и e-mail можно поступить и более радикально - удалить их из всех листингов. Плюсы: меньше полей в листинге, что потенциально сделает редактирование чуть проще. Но с другой стороны исчезнет уже существующая функциональность для тех, кто ею хотел бы воспользоваться. Я, например, вообще никогда ими не пользуюсь. Кому-нибудь они нужны? --Kiaora (обсуждение) 14:57, 11 июля 2017 (MSK)

Поле скайп было, по-моему, нашим собственным изобретением. Мы мало им пользовались, а сейчас, с появлением альтернативных сервисов, выделять именно скайп совсем уж странно, поэтому я не против того, чтобы избавиться от скайпа совсем. Адрес электронной почты может быть, видимо, актуален для гостиниц в глуши или в странах, где нет нормальных сервисов для онлайн-бронирования. --Alexander (обсуждение) 15:40, 11 июля 2017 (MSK)

Согласен, в Иране имейлы были очень нужны. --Kiaora (обсуждение) 15:56, 11 июля 2017 (MSK)

Полем skype никогда не пользовался, а электронная почта актуальна. Также предлагаю убрать значок рубля из поля «цена», так как мы решили этот значок не использовать. GMM (обсуждение) 15:49, 11 июля 2017 (MSK) Но из листинг-эдитора электронную почту действительно можно убрать. GMM (обсуждение) 15:50, 11 июля 2017 (MSK)

Хорошо, давайте удалим значок рубля, почту и скайп из листинг-эдитора. А скайп - из всех листингов. Подождём только, может быть, кто-то будет против. --Kiaora (обсуждение) 15:56, 11 июля 2017 (MSK)

Готово. Перенесите, пожалуйста, Участник:Kiaora/ListingEditor.js в MediaWiki:Gadget-ListingEditor.js. Спасибо.--Kiaora (обсуждение) 23:14, 31 июля 2017 (MSK)

Перенёс. Спасибо! --Alexander (обсуждение) 23:40, 31 июля 2017 (MSK)

Прекрасно. Но там были проблемы с сохранением format и email. Давайте попробуем ещё разок перенести? И можете ещё удалить поле Skype из новых листингов? --Kiaora (обсуждение) 01:10, 1 августа 2017 (MSK)

Сделал. --Alexander (обсуждение) 02:21, 1 августа 2017 (MSK)

ListingEditor перестал добавлять ссылку после основных заголовков

Судя по всему, немного поменялась html разметка: каждый h2 элемент теперь содержит 3 span <span id=".D0.9A.D0.B0.D0.BA_.D0.B4.D0.BE.D0.B1.D1.80.D0.B0.D1.82.D1.8C.D1.81.D1.8F"></span>, <span class="mw-headline" id="Как_добраться">Как добраться</span> и <span class="mw-editsection">..</span>. Поэтому нужно немного подправить скрипт: заменить sectionId = encodeURIComponent(sectionId).replace(/%20/g,'_').replace(/%/g,'.'); на sectionId = sectionId.replace(/ /g,'_'); --Kiaora (обсуждение) 22:23, 4 апреля 2018 (MSK)

Изменил. --Alexander (обсуждение) 23:25, 4 апреля 2018 (MSK)

Ссылка на карту

Кажется, нужно заменить http://maps.wikivoyage-ev.org/w/geomap.php на его аналог https://wikivoyage.toolforge.org/w/geomap.php - первый адрес пока в целом работает, но уже не поддерживается. В английском и немецком разделах это уже сделали. --Bok (обсуждение) 01:33, 28 января 2022 (MSK)

Заменил, спасибо. --Alexander (обсуждение) 10:40, 28 января 2022 (MSK)

локализация

Просьба в 81‐й строке поменять en на ru, в 89‐й — сделать троеточие единым (…), в 94‐й написа́ть «Файл», в 102‐й сделать ссылку прямой, а в 103‐й всё‐таки написа́ть латиницей. 217.117.125.83 22:04, 20 апреля 2022 (MSK)

Всё сделано кроме 102-й строки, поскольку я не совсем понял, что значит "сделать ссылку прямой". Поясните, пожалуйста. --Alexander (обсуждение) 00:04, 21 апреля 2022 (MSK)

@Atsirlin, там вторая ссылка ведёт на страницу в enwiki, которая перенаправляет в ruwiki. 217.117.125.83 19:36, 21 апреля 2022 (MSK)

Изменил, спасибо! -- Alexander (обсуждение) 00:23, 22 апреля 2022 (MSK)