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

New security helmet and coat #1586

Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open

New security helmet and coat #1586

wants to merge 56 commits into from

Conversation

Glamyrio
Copy link

@Glamyrio Glamyrio commented Oct 10, 2024 •
edited by m-dzianishchyts
Loading

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

ПР добавляет набор из трёх предметов которые можно примерить на вашей кукле.

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

Моя звай Гламур! Я люблю делай косплеи! Нраица! Эта моя государства Нанотразен. Он середина между СССП, ТСФ и ещё чиндикат. Здесь моя город Кибериада. Это Вульпикс, ерпшер местный. Нехороший! А вот здесь наш бриг. А здесь живёт СЕ с Поли, местный механько и см-о-дестабилизаторомахер. Это моя хата. Заходи, пожалуйста! Он моя сосед. Ассистос Серый. Боль в моя дырка задница. Я делай всё из стекла, он тоже делай всё из стекла. Я делай баню, он тоже делай баню. Я закупай косплей. Он не накопил. Радостно! (что только не придумаешь в 3 часа ночи)
это крутой костюм. Все.

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

screen1
screen2
screen3
Возможно в процессе придётся изменить спрайты

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

протестировано на локалке, все работает

Changelog

🆑 glamyr
add: Добавлен набор метрокопа в SecDrobe.
fix: Одежда младшего персонала теперь доступна в вендорах отделов.
/:cl:

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

AyIong commented Oct 10, 2024

Ты же говорил что это не на наш билд
Спрайт маски к слову, всё ещё лоу-контраст оверколоред шляпа, ещё и огромная
Ты бы со спрайтерами посоветовался

@AyIong AyIong requested a review from dj-34 October 10, 2024 00:12
@Glamyrio
Copy link
Author

Ты же говорил что это не на наш билд Спрайт маски к слову, всё ещё лоу-контраст оверколоред шляпа, ещё и огромная Ты бы со спрайтерами посоветовался

Я имел ввиду что спрайт не из этого билда, а насчет шляпы думаю зареспрайтить

PhantornRU
PhantornRU requested changes Oct 10, 2024
View reviewed changes
Copy link
Collaborator

@PhantornRU PhantornRU 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
image

dj-34
dj-34 reviewed Oct 10, 2024
View reviewed changes
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

спрайт не очень, и куртка какая-то обрывистая больно

@Glamyrio
Copy link
Author

Задумка нравится, но да, нужно поправить сами спрайты. Я б без шуток, но вообще сделал бы таких юнитов "РХБЗ-ЕРТ". Если хочется зайти дальше, то можешь буквально сделать аутфит РХБЗ-ЕРТ. Ну так, просто если захочешь :) Глазницы - точно нужно переспрайтить ИЛИ снизить прозрачность. Зайди в спрайтерскую и там это обсуди лучше, поможем.

Для начала чтобы зайти в спрайтерскую нужно получить роль, к тому-же у меня уже есть идеи как зареспрайтить их, а пока я буду экспериментировать. Насчёт одежды РХБЗ-ЕРТ точно не знаю, возможно, если пройдёт этот пр, затем можно будет заняться, но как я говорил, точно не знаю.

@Glamyrio
Copy link
Author

screenы
теперь линзы шлема прозрачны, а сам шлем смахивает на клуелес.
Да да, те пиксели на линзах это глаза. Так же насчёт куртки, у меня нет понятия что не так с ней, для меня она выглядит впорядке

@Glamyrio Glamyrio requested a review from PhantornRU October 10, 2024 19:24
@dj-34
Copy link
Collaborator

dj-34 commented Oct 11, 2024

screenы теперь линзы шлема прозрачны, а сам шлем смахивает на клуелес. Да да, те пиксели на линзах это глаза. Так же насчёт куртки, у меня нет понятия что не так с ней, для меня она выглядит впорядке

Не сказать что что-то поменялось. Все еще нет контрастного аутлайна, очень плоский подбор цветов у линз. Расовые спрайты.

@Glamyrio
Copy link
Author

ыскуут

@Glamyrio
Copy link
Author

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

@Glamyrio Glamyrio requested a review from dj-34 October 11, 2024 16:41
m-dzianishchyts
m-dzianishchyts requested changes Oct 13, 2024
View reviewed changes
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

протестировано на локалке, все работает

Нужен скрин с надетыми шмотками для всех доступных рас - уверен, что одного спрайта тут будет недостаточно

modular_ss220/clothing/code/helmet.dm Show resolved Hide resolved
@Glamyrio
Copy link
Author

протестировано на локалке, все работает

Нужен скрин с надетыми шмотками для всех доступных рас - уверен, что одного спрайта тут будет недостаточно

Вполне даже в порядке

@Glamyrio
Copy link
Author

У всех тех рас в списке такие же головы как у обычного хумана

@m-dzianishchyts
Copy link
Collaborator

У всех тех рас в списке такие же головы как у обычного хумана

Что насчет армора?

@Glamyrio
Copy link
Author

У всех тех рас в списке такие же головы как у обычного хумана

Что насчет армора?

Армор универсальный, смотрится на всех куклах нормально, кроме греев и воксов

m-dzianishchyts
m-dzianishchyts requested changes Oct 13, 2024
View reviewed changes
modular_ss220/clothing/code/clothing_vendors.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/clothing_vendors.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/helmet.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/helmet.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/suits.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/suits.dm Outdated Show resolved Hide resolved
Glamyrio and others added 2 commits October 14, 2024 01:29
@Glamyrio
Copy link
Author

Glamyrio commented Nov 8, 2024

2dludi

@Glamyrio
Copy link
Author

Glamyrio commented Nov 8, 2024

кпб нету, ибо нечего менять

@dj-34
Copy link
Collaborator

dj-34 commented Nov 9, 2024

2dludi

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

@Glamyrio
Copy link
Author

Glamyrio commented Nov 9, 2024

2dludi

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

Понял

@Glamyrio
Copy link
Author

Glamyrio commented Nov 9, 2024

2dchelovecki

@github-actions github-actions bot added the Merge Conflict Ну блять... label Nov 16, 2024
@Glamyrio
Copy link
Author

ахуеть

PhantornRU
PhantornRU approved these changes Nov 18, 2024
View reviewed changes
Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Спрайты интересные в любом случае

@Glamyrio
Copy link
Author

А конфликт можно убрать? Тут разработчик нужен

@github-actions github-actions bot removed the Merge Conflict Ну блять... label Nov 18, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Dec 3, 2024
@github-actions github-actions bot removed the Stale ПР долго был не активен, и требует обновления. label Dec 4, 2024
dj-34
dj-34 approved these changes Dec 8, 2024
View reviewed changes
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Нейм в английский переделай, на шлем похуй крч (но можно посмотреть кстати на билдах хл2 получше спрайтик. здесь например https://github.com/HL2RP-City-13/HL2RP-City-13
Но смотреть опять же на лицензию

@github-actions GitHub Actions
Copy link

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Dec 23, 2024
@dj-34 dj-34 added Stale Exempt PR не может быть устаревшим. Видать кто-то подкупил хэда. and removed Stale ПР долго был не активен, и требует обновления. labels Dec 24, 2024
@AyIong
Copy link
Collaborator

AyIong commented Dec 25, 2024

Почини линтер и можно мерге

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

@PhantornRU PhantornRU PhantornRU approved these changes

@dj-34 dj-34 dj-34 approved these changes

@m-dzianishchyts m-dzianishchyts m-dzianishchyts left review comments

Assignees
No one assigned
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Stale Exempt PR не может быть устаревшим. Видать кто-то подкупил хэда.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Glamyrio @AyIong @dj-34 @m-dzianishchyts @SadestMage @PhantornRU @ss220bot