-
Notifications
You must be signed in to change notification settings - Fork 242
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
Одна форма отправки ответов к комментариям #1216
Conversation
replaces a hidden form for each element with one form managed by ReplyContainer vue component
@@ -57,7 +57,6 @@ export default { | |||
data() { | |||
return { | |||
selectedUserIndex: null, | |||
postSlug: null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
эта штука не используется в компоненте
Супер, по-моему. Я не могу оценить код в темплейтах, но я оценил UX ручками. Руками баги не нашел. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Охуенно!
Размеры: |
опередил. ага, чуть меньше 10% в экономии. вроде и не много, но в количестве dom объектов там наверняка приличней. надо было бы ещё память замерять |
Что
Использует одну форму отправки ответов к комментариям вместо того, чтобы создавать скрытую форму ответа для каждого комментария
Демка в loom (звук так себе получился, сорян)
Зачем
Мы генерим много форм, из которых используется только одна. Ожидаем, что размер страницы уменьшится и браузеру станет полегче.
Сейчас есть такие темы:
Как
ReplyContainer
. Он содержит форму ответа на комментарии.Closes #1210