Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show in tag autocomplete that alias will be replaced #103

Open
7nik opened this issue Jan 14, 2020 · 20 comments
Open

Show in tag autocomplete that alias will be replaced #103

7nik opened this issue Jan 14, 2020 · 20 comments

Comments

@7nik
Copy link

7nik commented Jan 14, 2020

It would nice to show users that the searched tag will be replaced with another (usually with more correct naming).
It is how it done on Danbooru.
image

@stalkerg
Copy link
Owner

Main problem - it's a too long string for this widget.

@7nik
Copy link
Author

7nik commented Jan 18, 2020

Just make it multiline as a long tag name
image

Danbooru just makes the autocomplete wider, but we cannot because it can be located anywhere.

@stalkerg
Copy link
Owner

Ok, did it, please check it.

@7nik
Copy link
Author

7nik commented Jan 18, 2020

image
Looks quite spammy.

Weren't aliases at the end of the autocomplete list? Like tags with count = 1.

@MiTereKun
Copy link

MiTereKun commented Jan 18, 2020

Если абзацы в списке предложений заменили на полосочки, то и над первым вариантом нужна такая же.. Сразу заметил, что что-то не так. Хоть разница не сильно большая, но расстояние между полем ввода и первым вариантом стала ещё меньше и теперь они сильнее сливаются в белом стиле
image

@stalkerg
Copy link
Owner

Weren't aliases at the end of the autocomplete list? Like tags with count = 1

I made it the same as on Danbooru - ordering by original tag instead alias tag.

@stalkerg
Copy link
Owner

Looks quite spammy.

Can you check light theme?

@7nik
Copy link
Author

7nik commented Jan 18, 2020

image
The outline splits the input and the autocomplete, but the outline disappears (unlike the autocomplete) at right-click anywhere.

Also, now we have two lines for the same results.

@MiTereKun
Copy link

MiTereKun commented Jan 18, 2020

7nik, только хром выделяет строку ввода тегов, а на https://anime-pictures.net/pictures/view_posts/0?lang=ru ещё и крестик добавляет для полной очистки. Согласен, это создаёт полоску между списком и полем ввода, но фаерфокс (ни новый, ни старый) ничего подобного не делает.
Даже интересно стало - это именно движок браузера вытворяет, или специально прописанная фича на сайте
к примеру на http://cinemate.cc тоже только в хроме поле ввода выделяется синим

@7nik
Copy link
Author

7nik commented Jan 18, 2020

это часть базового хромовского CSS, если так можно сказать
image

@stalkerg
Copy link
Owner

Даже интересно стало - это именно движок браузера вытворяет, или специально прописанная фича на сайте

Это дефолтные стили хрома такие. Могу отключить к слову.

@stalkerg
Copy link
Owner

@7nik

Also, now we have two lines for the same results.

What you wanted was what you got. :)

@MiTereKun
Copy link

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

@7nik
Copy link
Author

7nik commented Jan 18, 2020

What you wanted

Like tags with count = 1

P.S.: What about allowing to display hidden and auto-tags in the search field?

@iDolmatov
Copy link
Contributor

То, что вы хотели, было тем, что вы получили. :)

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

Примеры:
М - мужчина \\ мальчик -> мужчина --- последний вариант исключить
Му \\ Ма --- уже без дубликатов, исключать ненужно

Явно можно убрать пару вариантов
1

Вообще выглядит так, словно "3 девушки" и "3 girls" разные теги, аналогично про мальчиков. И так далее.
2 3

@7nik
Copy link
Author

7nik commented Jan 19, 2020

@iDolmatov

"3 девушки" и "3 girls" разные теги

так есть, точнее "3 girls" ("3 девушки") и "3girls" (алиас, у этого тега нет перевода) это разные теги.

@iDolmatov
Copy link
Contributor

@7nik по сути то использоваться будет основной тег, поэтому показ перевода в этом случае не имеет смысла. Без перевода хоть понятно, что неважен выбор.
4

@stalkerg
Copy link
Owner

@iDolmatov т.е. ты предлагаешь сохранять язык тега? сложна....

@iDolmatov
Copy link
Contributor

iDolmatov commented Feb 29, 2020

@stalkerg при загрузке списка доступных тегов убирать алиасы при наличии в подсказке тегов, на которые ссылаются алиасы.

Например, получаем в списке теги с ID 109883 и 6. Видим, что 109883 пересылает на 6. Значит его ненужно показывать.
Сейчас подгружаются оба варианта.
image
Насколько понимаю, необходим ещё параметр parent. Не думаю, что сравнение до 10 вариантов слишком сложно. Проверяем наличие параметра parent у 10 ответов и найденные сравниваем с загруженными ID. При совпадении убираем проверенный тег из ответа. Вероятно, нужен будет какой-то промежуточный запрос-проверка: База тегов <> скрипт проверки <> форма запроса тегов.
Затруднение вижу только в том, чтобы дополнительная проверка не перезагружала сервер. Либо можно попробовать встроить проверку на стороне клиента, но, по моему мнению, это сложнее.
Другая мысль, что запрос должен проверять сначала наличие parent и при его отсутствии брать ID. ID имеет приоритет над parent и при совпадении тег со вторым параметром игнорируется. Однако, не уверен в возможности такой реализации.

@stalkerg
Copy link
Owner

stalkerg commented Mar 4, 2020

Спасибо. Я попробую что то с этим сделать. Увы в лоб тут нельзя.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants