fix(): mark battle comments as new #1084
Conversation
|
This pr is for #1070 |
|
Да хз, цветом он уже выделяется, а ломать стандартные цвета как-то не хочется |
|
Ну там раньше он был либо зеленым, либо синим, теперь для новых мнений становится серым, нормальным цветом для непрочитаного, при этом по цвету заголовка (синего, зеленого) понятно в какую сторону мнение. можно сломать для батлов это выделение серым для нового, благо его и не было раньше, и тогда я назову новый класс например comment-type-battle-is-new и поправлю arrowscroll и тогда arrowscroll будет переходить на новые мнения, но при этом внешне они будут выглядеть как раньше. а можно выделять серым как и все новые комменты, и тогда можно принять pr как есть. |
|
Без скриншотов непонятно :( |
|
|
|
Ну вот примерно такого я и ожидал. Некрасиво ломать первичную логику ради вторичной, не нравится :( |
to prevent breaking coloring logic
|
Ну значит поправил, теперь визуально все останется так как и было. |
Родительский коммент aka мнение в батлах, тоже должен выделяться. Из минусов текущего pr'ы, он ломает цвет этого самого мнения, нужно либо поправить стили тогда выходит, либо добавлять не comment-is-new а другой класс.
@vas3k как лучше сделать? потому что серые мнения в батлах это какой то минус к ux'у