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

add a new hos weapon judge 12 ga revolver #1443

Merged
merged 2 commits into from
Aug 31, 2024
Merged

Conversation

Namen-n
Copy link

@Namen-n Namen-n commented Aug 24, 2024

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

Новый нереальный револьвер ХОСа Judge с барабаном на 3 выстрела 12 калибра.
Замена мейновского пончика на балансный (верим) аналог анаконды.

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

Замена пончика на аналог анаконды чтоб уважаемые люди на ХОСе могли хвастать.

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

https://www.youtube.com/watch?v=JeessNVKJlk

image

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

Покатит.

Changelog

🆑
add: Легендарный пончик ХОСа заменен на новый револьвер Judge
/:cl:

@github-actions github-actions bot added 🔉 Звук Услада для ушей. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! labels Aug 24, 2024
@m-dzianishchyts
Copy link
Collaborator

Если это относится к баунти, то будет отклонено по причине, что еще не одобрено. Как я понимаю, обсуждение еще продолжается

@Namen-n
Copy link
Author

Namen-n commented Aug 27, 2024

К баунти не относится, с его автором не договаривался и там условия другие, так что личный фетиш. (модельку еще делаю нормальную)

@m-dzianishchyts
Copy link
Collaborator

модельку еще делаю нормальную

Тогда, пожалуйста, в драфт до окончания работы
image

@Namen-n Namen-n marked this pull request as draft August 27, 2024 11:10
@Namen-n
Copy link
Author

Namen-n commented Aug 27, 2024

так?

@@ -88,6 +88,27 @@
icon = 'modular_ss220/objects/icons/ammo.dmi'
icon_state = "44_box"

/obj/item/gun/projectile/revolver/reclinable/judge
name = "\improper Judge 12 GA"
desc = "Тяжёлый револьвер ружейного калибра. Несмотря на короткий ствол и высокую отдачу крайне эффективное оружие ближней дистанции."
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

Так почему бы не сделать тряску экрана при выстреле как у снайперки?

Copy link
Author

Choose a reason for hiding this comment

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

Тряска есть, хочешь больше?

Copy link
Collaborator

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 Aug 28, 2024

Выведешь из драфта как будет готово

@Namen-n Namen-n marked this pull request as ready for review August 28, 2024 16:19
@Namen-n
Copy link
Author

Namen-n commented Aug 28, 2024

вроде все более-менее (надеюсь пару часов переставления пикселей того стоили), ну еще можно еган раундстартовый убрать, но это сами решайте

@m-dzianishchyts
Copy link
Collaborator

Красная мушка делает его игрушечным. Думаю, надо ее в какой-то оттенок серого

@Namen-n
Copy link
Author

Namen-n commented Aug 28, 2024

Серая выглядит уныло, как и первая версия с серой палитрой. Реальная пушка, с которой взята идея, как раз с подобной мушкой. 2c04a253-f902-4ff1-a675-fe3597f17911

@Namen-n
Copy link
Author

Namen-n commented Aug 28, 2024

Как замержить это все добро (и мастер ветку тоже).image Потыкал туда сюда (в визулке), мб что то заруинил на локльной версии ну и ничего не понял.

@m-dzianishchyts
Copy link
Collaborator

Как замержить это все добро

Больше ничего делать не нужно, это отставание в истории ни на что не влияет, конфликтов нет, мержить мастер нет нужды.
Теперь ждать решение разработчиков-коллабораторов

@Namen-n
Copy link
Author

Namen-n commented Aug 28, 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.

Выглядит норм
Только чейнджлог бы поправить, не обязательно и add и del
Можно всё в одном add уместить. Мол это заменило это

@Namen-n
Copy link
Author

Namen-n commented Aug 29, 2024

Ну как то так

@AyIong AyIong requested a review from Legendaxe August 29, 2024 05:45
@dj-34
Copy link
Collaborator

dj-34 commented Aug 29, 2024

и все таки красная мушка выглядит не очень уместно

@dj-34
Copy link
Collaborator

dj-34 commented Aug 29, 2024

Еще хочется сказать касаемо звука. Мне кажется такой звук уже есть в билде, либо похожий. Ну и не сильно он похож на "дробашный" выстрел.

@Namen-n
Copy link
Author

Namen-n commented Aug 29, 2024

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

@Legendaxe Legendaxe merged commit ce2ec8a into ss220club:master Aug 31, 2024
16 checks passed
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.

6 participants