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

[FIX] Gun rack parts fix #1400

Merged
merged 16 commits into from
Aug 1, 2024
Merged

[FIX] Gun rack parts fix #1400

merged 16 commits into from
Aug 1, 2024

Conversation

skifsky
Copy link

@skifsky skifsky commented Jul 30, 2024

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

Добавляет иконку деталям оружейной стойки и возможность ее строительства

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

Оружейная стойка больше не оставляет после себя невидимые детали

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

Снимок экрана 2024-08-01 152831

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

Потыкал разным оружием и айтемами в разных интентах, собрал и разобрал, повторил то же самое на борге.

Changelog

🆑
fix: Оружейную стойку теперь возможно построить
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Jul 30, 2024
Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Переведи всякие уведомления в спанах,
Можешь ещё описания перевести

@AyIong
Copy link
Collaborator

AyIong commented Jul 30, 2024

imageadd: Добавлена иконка деталей для оружейной стойки это не обязательно

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

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

modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
@AyIong AyIong added the :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. label Jul 30, 2024
@github-actions github-actions bot removed the :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. label Jul 30, 2024
@dj-34 dj-34 self-requested a review July 30, 2024 14:17
@dj-34 dj-34 added the 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах label Jul 30, 2024
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
modular_ss220/aesthetics/racks/code/racks.dm Outdated Show resolved Hide resolved
@dj-34
Copy link
Collaborator

dj-34 commented Jul 30, 2024

Еще мне не нравится несоответствие в спрайте. Не хватает "отсвечивания" и внутренняя часть стеллажа окрашена в иной цвет.

И ножки... какие то жирные, для сборного стеллажа... Можно на 1 пиксель ужать, чтобы был по толщине как у gunrack'а.

image

@github-actions github-actions bot removed the 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах label Jul 31, 2024
@AyIong
Copy link
Collaborator

AyIong commented Jul 31, 2024

Сюда жамкай как всё сделаешь
image

@skifsky skifsky requested a review from AyIong July 31, 2024 14:16
@dj-34
Copy link
Collaborator

dj-34 commented Jul 31, 2024

Спрайт то может и лучше стал, но цветовая палитра все еще не та. Подгони "белесый" цвет к синюшнему с оригинала - в первой итерации примерно то что нужно было.

image

@skifsky
Copy link
Author

skifsky commented Aug 1, 2024

Оставлю только фикс деталей оружейной стойки, потому что там чет пиздец рантаймы идут. Может быть потом отдельным пиаром уже сделаю...

@skifsky skifsky requested a review from dj-34 August 1, 2024 11:49
@skifsky skifsky changed the title Add new rack & gun rack parts fix [FIX] Gun rack parts fix Aug 1, 2024
@Gaxeer Gaxeer merged commit deb642c into ss220club:master Aug 1, 2024
14 checks passed
@skifsky skifsky deleted the rack-shelving branch August 2, 2024 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants