-
Notifications
You must be signed in to change notification settings - Fork 155
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
Анимации крыльев Нианов #719
Conversation
Медиа точно надо бы |
Реди |
Я не спрайтер, но в кнопку бы другой спрайт. |
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.
WavingSystem, во-первых, лежит не в нашей папке, про копирайт говорить и смысла нет. Во-вторых, очевидно, система скопипащена. Надо использовать либо оригинальную систему, либо сделать помарку, что система взята откуда-то и т. п. (но чистый копипаст всё равно использовать нельзя).
Ревью по спрайтам
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Ваше мнение для нас очень важно, но не очень то и нужно |
Как жаль, что вы считаете моё мнение не нужным, ведь для мержа ПРа все пункты ревью должны быть выполнены. |
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.
Ревью по спрайтам
Дублирую в ревью, раз не очень ясно.
- Нужно исправить иконку действия
- Исправить линейность анимации
Линейные анимации свойственны машинерии, однако в анимации живых существ должно быть видно нелинейное движение.
В анимации хвостов унатхов это сделано благодаря количеству спрайтов в анимации, наложенных на 0.1 задержку. Тут же и малое количество спрайтов анимации, и большая её задержка. - Если и добавлять такую фичу - то на все виды крыльев.
- Не стоит добавлять эту анимацию, как перманентную. Это пипец как странно выглядит на кошачьих хвостах на трупах, т.к. трупы не способны к движению, а анимки у мертвого тела не останавливаются. В этом случае это будет выглядеть ещё хуже, т.к. область анимации ещё больше.
Либо исправляет замечания, либо закрываем, @DexlerXD ? |
ну, продвижений не было, исправлять тут не вижу смысла |
Описание PR
Медиа
Loader.2024.02.07.-.15.57.25.03.mp4
Проверки
Изменения
🆑