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: Новые прически #1447

Merged
merged 8 commits into from
Sep 1, 2024
Merged

Add: Новые прически #1447

merged 8 commits into from
Sep 1, 2024

Conversation

Sirstaniya
Copy link

@Sirstaniya Sirstaniya commented Aug 26, 2024

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

Добавляет 23 новые прически для людей, слаймоменов, КПБ. Некоторые доступны таярам и 1 доступна только таярам (есть альтернатива для остальных).

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

Разнообразие

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

image
image
image
image
image
image
image
image
image
image
upd:
изображение
изображение
изображение
изображение
изображение
изображение
изображение
изображение
изображение
изображение
изображение

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

Запустил, проверил, сделал скрины.

Changelog

🆑
imageadd: Добавил новые прически
/:cl:

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

С этими прическами как будто лицо должно быть частично видно в профиль
image
image

Типа как тут
image

@m-dzianishchyts
Copy link
Collaborator

Тег в чейнджлоге надо поменять на imageadd

@Sirstaniya
Copy link
Author

Sirstaniya commented Aug 26, 2024

Сильно критично с видом сбоку?

@m-dzianishchyts
Copy link
Collaborator

Сильно критично с видом сбоку?

Ну я не спрайтер, но проблема явно есть

@Sirstaniya
Copy link
Author

Исправил спрайты двух причесок, чейнджлог на всякий случай изменил (Знать бы что вместо edit указать, ибо не совсем понимаю)

@m-dzianishchyts
Copy link
Collaborator

Знать бы что вместо edit указать

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

@Sirstaniya
Copy link
Author

исправил

Comment on lines 547 to 550
///////////////////////////////////
/////////OT Sirstaniya////////////
//////////////////////////////////
/* New human hair */
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
///////////////////////////////////
/////////OT Sirstaniya////////////
//////////////////////////////////
/* New human hair */

Эти комментарии можно убирать - по истории коммитов будет известно, кто автор

icon = 'modular_ss220/hairs/icons/human_hair.dmi'
name = "Two Side Up"
icon_state = "twosideup"
species_allowed = list("Human", "Slime People")
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.

image
image
Залил прически под таяр. IPC не добавлял, так как им и так доступны прически людей. Так же на вульп прически не подходят, ибо нужно спрайтить их под морды. Пойдет?

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.

чет пиздец прича для таяр, имхо

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.

Эстетическую составляющую тоже стоит учитывать

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
Author

Choose a reason for hiding this comment

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

Сейчас погляжу, поправлю. Тут еще выразили желание наспрайтить пару вариантов, поэтому буду еще и этим заниматься.

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.

Больше нечего добавить

@Sirstaniya Sirstaniya marked this pull request as draft August 29, 2024 09:25
Оставим таярам более короткие и сдержанные прически.
@Sirstaniya Sirstaniya marked this pull request as ready for review August 31, 2024 13:31
Добавлено еще несколько причесок.
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.

Здесь высоты профиля и анфаса не совпадают.

image
image
image

Эту синюю штуку точно должно быть видно в профиль. Вообще она выглядит очень чужеродно.

image

Тут пикселя будто в профиль не хватает, странная впадина.

image

@Sirstaniya
Copy link
Author

Sirstaniya commented Aug 31, 2024

Здесь высоты профиля и анфаса не совпадают.

image image image

Эту синюю штуку точно должно быть видно в профиль. Вообще она выглядит очень чужеродно.

image

Тут пикселя будто в профиль не хватает, странная впадина.

image

Поправил высоты и недостающие элементы.
изображение
изображение
изображение
изображение

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

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.

Я поверил короче

@dj-34 dj-34 changed the title Новые прически Add: Новые прически Sep 1, 2024
@dj-34 dj-34 merged commit e1d2d6f into ss220club:master Sep 1, 2024
13 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