Skip to content

WIP Feature/177414 mc-tags #112

Merged
merged 17 commits into from
Apr 24, 2019
Merged

WIP Feature/177414 mc-tags #112

merged 17 commits into from
Apr 24, 2019

Conversation

lskramarov
Copy link
Contributor

Добавлен autocomplete и tag-list.

Смотреть server-dev:autocomplete и server-dev:tags.

Мне осталось добавить тесты и посмотреть темную тему.

Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Давай в пример нормальные айтемы вставим?
Вот:

options = ['One', 'Two', 'Three', 'Four', 'Five', 'Longest text (0123456789 qwertyuiopasdfghjklzxcvbnm)',
    'Волгоград', 'Воронеж', 'Ейск', 'Екабпилс', 'Екатеринбург', 'Екатериновка', 'Екатеринославка', 'Екаша',
    'Екибастуз', 'Екпинди', 'Елань', 'Елец', 'Казань', 'Краснодар', 'Красноярск', 'Москва',
    'Нижний Новгород', 'Новосибирск', 'Омск', 'Пермь', 'Ростов-на-Дону',  'Самара', 'Санкт-Петербург',
    'Уфа', 'Челябинск'];

Поле тегов:

  1. http://d.mikeozornin.ru/FmLu7c
    Схлопывается, если удалить все теги

  2. http://d.mikeozornin.ru/nyj58d
    Системный шрифт и мелкий кегль внутри поля ввода

  3. http://d.mikeozornin.ru/nyj58d
    После удаления последнего тега поле теряет фокус.

  4. http://d.mikeozornin.ru/uCsGAe
    Вроде создать в этом случае не должно быть?

  5. http://d.mikeozornin.ru/LL2nlY
    После удаления последнего тега в автокомплите теряется фокус с поля, показывается на списке, но фокуса там на самом деле нет. ↓↑ двигает окно браузера

  6. http://d.mikeozornin.ru/Le3ZXo
    Надо куда-то записать обработку вставки

  7. http://d.mikeozornin.ru/Bh2KVP
    В синих же должен быть синий текст, а в красных — красный?

  8. http://d.mikeozornin.ru/X9ctwk
    Длинный тег кукожит себя и поле. Причем тег другой длины ок — http://d.mikeozornin.ru/PIUdMz
    Кукожить может в разных вариантах.

Пример тегов:

  1. http://d.mikeozornin.ru/M2HmZU
    Давай вот такие перебивки тоже брать из наших же стилей, чтобы не было ощущения, что сломался стиль.

@lskramarov
Copy link
Contributor Author

  1. http://d.mikeozornin.ru/uCsGAe
    Вроде создать в этом случае не должно быть?
  2. http://d.mikeozornin.ru/Le3ZXo
    Надо куда-то записать обработку вставки

это ограничения, Я о них говорил на прошлом синке.

Остальное поправлю.

@lskramarov lskramarov force-pushed the feature/177414-typeahead branch from 6ec28d1 to 839f91d Compare April 17, 2019 13:27
@mikeozornin
Copy link
Contributor

http://d.mikeozornin.ru/0cwso4
Вот этот текст — плейсхолдер, он должен быть бледней. Тут http://d.mikeozornin.ru/ZHNoV2 норм.

Я заапрувлю, тут нечего смотреть ещё раз.

Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@lskramarov lskramarov force-pushed the feature/177414-typeahead branch from 419889a to d87d1e0 Compare April 22, 2019 14:50
@lskramarov lskramarov changed the title WIP Feature/177414 typeahead WIP Feature/177414 mc-tags Apr 23, 2019
@lskramarov lskramarov merged commit 76a98ab into master Apr 24, 2019
pimenovoleg pushed a commit that referenced this pull request May 7, 2019
@pimenovoleg pimenovoleg deleted the feature/177414-typeahead branch June 8, 2019 09:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants