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

Анимации крыльев Нианов #719

Closed
wants to merge 5 commits into from

Conversation

Xenoksys
Copy link

@Xenoksys Xenoksys commented Feb 7, 2024

Описание PR

Медиа

Loader.2024.02.07.-.15.57.25.03.mp4

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • add: Анимации крыльев нианов

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Sprites Изменение затронуло файлы ".rsu / .png" labels Feb 7, 2024
Copy link

github-actions bot commented Feb 7, 2024

RSI Diff Bot; head commit 4105195 merging into 9c0a6d5
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Mobs/Customization/moth-anim.rsi

State Old New Status
tail_striped_wagging Added

Edit: diff updated after 4105195

@SkaldetSkaeg
Copy link

Медиа точно надо бы

@Xenoksys
Copy link
Author

Xenoksys commented Feb 7, 2024

Медиа точно надо бы

Реди

@SkaldetSkaeg
Copy link

Я не спрайтер, но в кнопку бы другой спрайт.

Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

WavingSystem, во-первых, лежит не в нашей папке, про копирайт говорить и смысла нет. Во-вторых, очевидно, система скопипащена. Надо использовать либо оригинальную систему, либо сделать помарку, что система взята откуда-то и т. п. (но чистый копипаст всё равно использовать нельзя).

@WhitlyOfficial
Copy link

Ревью по спрайтам

  1. Нужно исправить иконку действия
  2. Исправить линейность анимации
    Линейные анимации свойственны машинерии, однако в анимации живых существ должно быть видно нелинейное движение.
    В анимации хвостов унатхов это сделано благодаря количеству спрайтов в анимации, наложенных на 0.1 задержку. Тут же и малое количество спрайтов анимации, и большая её задержка.
  3. Если и добавлять такую фичу - то на все виды крыльев.
  4. Не стоит добавлять эту анимацию, как перманентную. Это пипец как странно выглядит на кошачьих хвостах на трупах, т.к. трупы не способны к движению, а анимки у мертвого тела не останавливаются. В этом случае это будет выглядеть ещё хуже, т.к. область анимации ещё больше.

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Feb 11, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Xenoksys
Copy link
Author

Ревью по спрайтам

1. Нужно исправить иконку действия

2. Исправить линейность анимации
   Линейные анимации свойственны машинерии, однако в анимации живых существ должно быть видно нелинейное движение.
   В анимации хвостов унатхов это сделано благодаря количеству спрайтов в анимации, наложенных на 0.1 задержку. Тут же и малое количество спрайтов анимации, и большая её задержка.

3. Если и добавлять такую фичу - то на все виды крыльев.

4. Не стоит добавлять эту анимацию, как перманентную. Это пипец как странно выглядит на кошачьих хвостах на трупах, т.к. трупы не способны к движению, а анимки у мертвого тела не останавливаются. В этом случае это будет выглядеть ещё хуже, т.к. область анимации ещё больше.

Ваше мнение для нас очень важно, но не очень то и нужно
Людям понравилось

@WhitlyOfficial
Copy link

Ревью по спрайтам

1. Нужно исправить иконку действия

2. Исправить линейность анимации
   Линейные анимации свойственны машинерии, однако в анимации живых существ должно быть видно нелинейное движение.
   В анимации хвостов унатхов это сделано благодаря количеству спрайтов в анимации, наложенных на 0.1 задержку. Тут же и малое количество спрайтов анимации, и большая её задержка.

3. Если и добавлять такую фичу - то на все виды крыльев.

4. Не стоит добавлять эту анимацию, как перманентную. Это пипец как странно выглядит на кошачьих хвостах на трупах, т.к. трупы не способны к движению, а анимки у мертвого тела не останавливаются. В этом случае это будет выглядеть ещё хуже, т.к. область анимации ещё больше.

Ваше мнение для нас очень важно, но не очень то и нужно Людям понравилось

Как жаль, что вы считаете моё мнение не нужным, ведь для мержа ПРа все пункты ревью должны быть выполнены.

Copy link

@WhitlyOfficial WhitlyOfficial left a comment

Choose a reason for hiding this comment

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

Ревью по спрайтам

Дублирую в ревью, раз не очень ясно.

  1. Нужно исправить иконку действия
  2. Исправить линейность анимации
    Линейные анимации свойственны машинерии, однако в анимации живых существ должно быть видно нелинейное движение.
    В анимации хвостов унатхов это сделано благодаря количеству спрайтов в анимации, наложенных на 0.1 задержку. Тут же и малое количество спрайтов анимации, и большая её задержка.
  3. Если и добавлять такую фичу - то на все виды крыльев.
  4. Не стоит добавлять эту анимацию, как перманентную. Это пипец как странно выглядит на кошачьих хвостах на трупах, т.к. трупы не способны к движению, а анимки у мертвого тела не останавливаются. В этом случае это будет выглядеть ещё хуже, т.к. область анимации ещё больше.

@DexlerXD DexlerXD marked this pull request as draft February 14, 2024 18:17
@Surani1
Copy link

Surani1 commented May 22, 2024

@Kit0vras @DexlerXD @Xenoksys что с крыльями по итогу?

@Kit0vras
Copy link

Либо исправляет замечания, либо закрываем, @DexlerXD ?

@DexlerXD
Copy link

ну, продвижений не было, исправлять тут не вижу смысла
если надо, кто-то другой сделает заново

@DexlerXD DexlerXD closed this May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Merge Conflict Some content has conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants