-
Notifications
You must be signed in to change notification settings - Fork 102
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
Add: Новые прически #1447
Conversation
Тег в чейнджлоге надо поменять на |
Сильно критично с видом сбоку? |
Ну я не спрайтер, но проблема явно есть |
Исправил спрайты двух причесок, чейнджлог на всякий случай изменил (Знать бы что вместо edit указать, ибо не совсем понимаю) |
Ничего можно не указывать, чейнджлог относится ко всему ПРу, а ПР добавляет новые прически, история изменений тут не нужна |
исправил |
/////////////////////////////////// | ||
/////////OT Sirstaniya//////////// | ||
////////////////////////////////// | ||
/* New human hair */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/////////////////////////////////// | |
/////////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") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Стоит пересмотреть допустимые расы, возможно, эти прически еще кому-то подойдут
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Как вообще получилось прическу повесить на кпб? они не должны быть доступны
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
они всегда были доступны и считаю это шикарным
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
чет пиздец прича для таяр, имхо
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ну, меня попросили пересмотреть "Подходят ли они". Если на вульп не подходит из-за морды, то на таяр уже чисто из-за вкусов как по мне
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Эстетическую составляющую тоже стоит учитывать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
У нас на билде вроде как множество причесок, которые так или иначе не подходят некоторым расам, но на таяр стоит доступ к волосам такой же, как и у людей (Если не ошибаюсь). Если все же является небольшой проблемой, готов изменить доступ к неподходящим волосам
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сейчас погляжу, поправлю. Тут еще выразили желание наспрайтить пару вариантов, поэтому буду еще и этим заниматься.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Больше нечего добавить
Оставим таярам более короткие и сдержанные прически.
Добавлено еще несколько причесок.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я поверил короче
Что этот PR делает
Добавляет 23 новые прически для людей, слаймоменов, КПБ. Некоторые доступны таярам и 1 доступна только таярам (есть альтернатива для остальных).
Почему это хорошо для игры
Разнообразие
Изображения изменений
upd:
Тестирование
Запустил, проверил, сделал скрины.
Changelog
🆑
imageadd: Добавил новые прически
/:cl: