В PR была добавлена анимация для нового уведомления о теме. В целом это выглядит хорошо, но по ошибке нарушен исходный механизм выравнивания.
Коренная причина заключается в том, что transform: translateX(50%); не работает вместе с новым CSS-свойством анимации animation: float-down 250ms ease-in-out forwards;. При активации анимации исходный transform перестает действовать.
Эту ошибку можно увидеть на приложенном файле.
1 лайк
Поскольку transform и анимация float-down конфликтуют друг с другом, я считаю, что есть два способа решить эту проблему:
Добавить новый класс анимации float-down-center в discourse.scss, включив transform: translateX(50%); в ключевые кадры.
Изменить способ выравнивания кнопки без использования атрибута transform, применив width: fit-content, left: 0, right: 0 и margin: auto;.
Если это так, я создам PR для исправления этого.
1 лайк
Здесь применяется PR, используя второй метод:
main ← small-lovely-cat:fix-misalignment-new-topic
merged 10:23PM - 18 Dec 25 UTC
In #36726, a new animation is added to the new topic notifictaion, however, it a… ccidentally break the center alignment before.
The root cause is that the `transform` attribute is also used in the `animation: float-down`, that overrides and invalidates the original `transform: translateX(50%)`.
This PR changes how the element is aligned, so as to fix the alignment problem and remove the original usage of `transform`.
Before:
<img width="1914" height="915" alt="image" src="https://github.com/user-attachments/assets/4dd0ad3d-b0d9-4526-ab18-2d47da69eb28" />
After:
<img width="1916" height="920" alt="image" src="https://github.com/user-attachments/assets/72af14b1-eb39-432a-9131-2cb38a8c557d" />
2 лайка
chapoi
18.Декабрь.2025 08:20:57
7
@jordan.vidrine , я тоже подумал, что заметил это
Внесены небольшие исправления, скоро объединим.
2 лайка
Вместо этого я объединил ваш PR. Спасибо!
1 лайк