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

Листаем комменты в батлах с помощью кнопки comment-scroll-arrow #1267

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

trin4ik
Copy link
Contributor

@trin4ik trin4ik commented Nov 13, 2024

По просьбам трудящихся
https://vas3k.club/post/26312/#comment-0fc85b28-b8b8-456e-aeb2-694f0572cb4b

Основная проблема в том, что по логике CommentScrollArrow.vue мы листаем комменты/реплаи, у которых есть атрибут id^=comment-. В батлах же атрибут id="comment-bla-bla-bla" у родительского элемента. Судя по всему, это сделано из-за наличия доп. элементов внутри "основного коммента". Однако CommentScrollArrow.vue так не работает и всё ломается.
Перенёс идентификатор к элементу class=comment, а так же пришлось немного поверстать.

Подробнее по коду в ревью пробегусь.
В ЛИСЕ ПРОВЕРИЛ!!!

@trin4ik trin4ik requested a review from vas3k as a code owner November 13, 2024 20:15
<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 }}">
Copy link
Contributor Author

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">
Copy link
Contributor Author

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;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

+20px к scroll-margin-top, т.к. при переходе на якорь в батлах надо добавить отступ для отображения за что оставлен коммент
image

@@ -1016,9 +1016,17 @@
grid-template-areas:
"prefix prefix prefix"
"left center right"
"reply-form reply-form reply-form"
Copy link
Contributor Author

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;
Copy link
Contributor Author

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",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

к общему массиву новых комментов добавляем батловые реплаи первого и второго уровня

@trin4ik
Copy link
Contributor Author

trin4ik commented Nov 13, 2024

Кстати, в CommentScrollArrow.vue

const highlightComment = () => {

запускается анимация, которая ничем не описана, я не нашёл. т.е. как-будто что-то планировалось и не сделалось. могу убрать заодно.

Copy link
Owner

@vas3k vas3k left a comment

Choose a reason for hiding this comment

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

Анимацию пока не трогай, хз откуда она :)

@vas3k vas3k merged commit d004535 into vas3k:master Nov 13, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

2 participants