-
Notifications
You must be signed in to change notification settings - Fork 240
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
Листаем комменты в батлах с помощью кнопки comment-scroll-arrow
#1267
Conversation
<div class="battle-comment-prefix battle-comment-prefix-side-{{ comment.metadata.battle.side }}"> | ||
за «{{ comment.battle_side }}» | ||
</div> | ||
<div class="block comment comment-type-battle comment-type-battle-side-{{ comment.metadata.battle.side }}"> | ||
<div class="block comment {% if comment.created_at > post_last_view_at %}comment-is-new{% endif %} comment-type-battle comment-type-battle-side-{{ comment.metadata.battle.side }}" id="comment-{{ comment.id }}"> |
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.
Перекинул id сюда, так же добавил класс comment-is-new
при необходимости
{% endfor %} | ||
</div> | ||
{% endif %} | ||
<div class="comment-replies thread-collapse-toggle"> |
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.
контейнер comment-replies
чтобы не ломать общие стили
@@ -418,6 +418,10 @@ | |||
|
|||
.comment-type-battle {} | |||
|
|||
.comment-type-battle.comment { | |||
scroll-margin-top: 40px; |
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.
@@ -1016,9 +1016,17 @@ | |||
grid-template-areas: | |||
"prefix prefix prefix" | |||
"left center right" | |||
"reply-form reply-form reply-form" |
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.
форма реплаев сразу под первым батловым комментом
@@ -583,6 +583,10 @@ | |||
justify-self: stretch; | |||
} | |||
|
|||
.comment-layout-battle .comment-replies { | |||
grid-area: replies; |
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.
а вот это, кажется, лишнее
// Новые реплаи на втором уровне бэтлов | ||
".battle-comments .post-comments-list > .replies > .reply:not(.comment-is-new) > .reply-replies >.replies > .reply.comment-is-new", | ||
".battle-comments-list .comment-replies > .replies > .reply:not(.comment-is-new) > .reply-replies >.replies > .reply.comment-is-new", |
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.
к общему массиву новых комментов добавляем батловые реплаи первого и второго уровня
Кстати, в
запускается анимация, которая ничем не описана, я не нашёл. т.е. как-будто что-то планировалось и не сделалось. могу убрать заодно. |
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.
Анимацию пока не трогай, хз откуда она :)
По просьбам трудящихся
https://vas3k.club/post/26312/#comment-0fc85b28-b8b8-456e-aeb2-694f0572cb4b
Основная проблема в том, что по логике
CommentScrollArrow.vue
мы листаем комменты/реплаи, у которых есть атрибутid^=comment-
. В батлах же атрибутid="comment-bla-bla-bla"
у родительского элемента. Судя по всему, это сделано из-за наличия доп. элементов внутри "основного коммента". ОднакоCommentScrollArrow.vue
так не работает и всё ломается.Перенёс идентификатор к элементу
class=comment
, а так же пришлось немного поверстать.Подробнее по коду в ревью пробегусь.
В ЛИСЕ ПРОВЕРИЛ!!!