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.

Sign up for GitHub

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

Jump to bottom

Добавляет сакуру, декаль листы на земле, тайлы травы с листьями и анимацию листопада #964

Closed
wants to merge 6 commits into from
Closed

Добавляет сакуру, декаль листы на земле, тайлы травы с листьями и анимацию листопада #964

wants to merge 6 commits into from

Conversation

RuslanM
Copy link

@RuslanM RuslanM commented Jan 28, 2024 •
edited
Loading

Что этот PR делает

Добавляет модуль, который содержит:

  1. Сакуру, листопад, листья на земле, траву с листьями, анимацию листопада
  2. dmi файлы со спрайтами для вещей из пункта 1

Также PR содержит вопросы по написанию тех методов, что я сам не смог

Почему это хорошо для игры

Лично я хочу в проекте сакуру и анимацию листопада

Изображения изменений

dreamseeker_3VeXG7GE6h

Тестирование

Пока только отображение спрайтов на локалке проверил

Changelog

🆑
add: сакуру и сопутствующие объекты
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Jan 28, 2024
@ss220bot ss220bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Jan 28, 2024
@RuslanM
Copy link
Author

RuslanM commented Jan 28, 2024 •
edited
Loading

Привет. RP на текущий момент не готов к финальном ревью. Мне нужна помощь к некоторыми методами, надеюсь, найти советы в комментариях.
Уточню, что готовые решения приветствуются, но если вы просто пример подкинете подходящий или ссылку на нужную часть учебника - это тоже в плюс.

AyIong
AyIong reviewed Jan 28, 2024
View reviewed changes
modular_ss220/flora/_flora.dm Outdated Show resolved Hide resolved
modular_ss220/flora/code/effects/blossom.dm Outdated Show resolved Hide resolved
dj-34
dj-34 reviewed Jan 29, 2024
View reviewed changes
Copy link
Collaborator

@dj-34 dj-34 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image

Вынеси в modular_ss220/objects/flora

RuslanM
RuslanM commented Jan 29, 2024
View reviewed changes
Copy link
Author

@RuslanM RuslanM 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. Изменил слой у анимации

@Legendaxe Legendaxe added the Нужна помощь Очень... ОЧЕНЬ нужна помощь! label Jan 29, 2024
@AyIong AyIong marked this pull request as draft January 31, 2024 18:07
@github-actions github-actions bot added the Merge Conflict Ну блять... label Feb 8, 2024
@AyIong
Copy link
Collaborator

AyIong commented Feb 26, 2024

Больше месяца и никаких сподвижек, откроешь по новой если решишь продолжить

@AyIong AyIong closed this Feb 26, 2024
@RuslanM
Copy link
Author

RuslanM commented Mar 2, 2024

Больше месяца и никаких сподвижек, откроешь по новой если решишь продолжить

Ага, я сейчас после операции восстанавливаюсь, как будут силы, вернусь к ПРу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AyIong AyIong AyIong left review comments

@dj-34 dj-34 dj-34 left review comments

Assignees
No one assigned
Labels
Merge Conflict Ну блять... 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Этот чейнджлог будет успешно опубликован Нужна помощь Очень... ОЧЕНЬ нужна помощь!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@RuslanM @AyIong @dj-34 @Legendaxe @ss220bot