search

eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Implemented displaying and editing tags #24

Closed egor-bogomolov closed 7 years ago

egor-bogomolov commented 7 years ago

Добавлены возможность добавлять и удалять теги в MomentEditor и их отображение в MomentView.

egor-bogomolov commented 7 years ago

Я попробовал версию с крестиком справа на TextView, при которой весь TextView остается кликабельным (действительно нашлась готовая библиотека bootstrap для этого), но тогда перестает быть понятно, почему кликабелен не только крестик, а еще и вся строчка. Поэтому я сделал версию, в которой у каждой строчки есть кнопка, по которой она удаляется.

krinkinmu commented 7 years ago

Еще раз, строка должна выглядеть как кнопка - что как бы намекает, что на нее можно нажать.

krinkinmu commented 7 years ago

А целая строка + крестик не решает проблему с неэффективным использованием места на экране, который вообще-то может быть и не очень большим.

egor-bogomolov commented 7 years ago

Давайте вернемся к решению проблемы использования места

Лучше бы использовать что-то такое: https://github.com/mcharmas/android-tagview/blob/master/library/src/main/java/pl/charmas/android/tagview/TagView.java , но только на базе EditText.

Здесь вывод тегов выглядит хорошо, зачем делать их на базе EditText? Поскольку теги это все-такие короткие строчки, наличие редактирования отдельного тега не выглядит слишком полезным.

krinkinmu commented 7 years ago

Нет, имелось ввиду, не EditText на каждый тег, а вместе с нарисованными тегами еще дополнительно добавить EditText, чтобы добавлять новые теги. Ну вроде набрали название в EditText нажали на ввод и новый тег автоматически добавляется. Чтобы удалить тег, просто тыкаем в тег.

egor-bogomolov commented 7 years ago

Нет, имелось ввиду, не EditText на каждый тег, а вместе с нарисованными тегами еще дополнительно добавить EditText, чтобы добавлять новые теги. Ну вроде набрали название в EditText нажали на ввод и новый тег автоматически добавляется. Чтобы удалить тег, просто тыкаем в тег.

Так действительно хорошо.

https://github.com/mcharmas/android-tagview

А можно ли эту библиотеку как-то импортировать, не копируя целиком? Я до этого всегда встречал в Readme строчку для добавления в gradle, а тут ее нет.

krinkinmu commented 7 years ago

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

krinkinmu commented 7 years ago

Вроде ок - можно мержить, только кажется, что EditText лучше под тегами, а не над ними.