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

fix(): enable go to second level comments in battles #1079

Merged
merged 3 commits into from
Apr 6, 2023

Conversation

zyuhel
Copy link
Contributor

@zyuhel zyuhel commented Apr 3, 2023

#1070 In battles second level comments in fact are third levels, and go to new message skips them.

@zyuhel zyuhel requested a review from vas3k as a code owner April 3, 2023 19:27
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.

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

@zyuhel
Copy link
Contributor Author

zyuhel commented Apr 3, 2023

Тут момент в том что в battle'ах в принципе другая верстка, за счет того что у нас первый уровень комментов это не то чтобы комменты, это мнения, за ту или иную сторону, могу добавить заодно какой нибудь класс к самим батлам чтобы можно было только для них это сделать.

@vas3k
Copy link
Owner

vas3k commented Apr 3, 2023

Там другая верстка просто потому что на нее забили болт. Добавить класс к верстке балтов — вот это правильный путь, да

@zyuhel zyuhel requested a review from vas3k April 3, 2023 20:16
@vas3k vas3k merged commit 7acedfb into vas3k:master Apr 6, 2023
@zyuhel zyuhel deleted the patch-4 branch April 6, 2023 15:18
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