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

New Hair v2.4 by WP #13523

Merged
merged 5 commits into from
Sep 24, 2024
Merged

New Hair v2.4 by WP #13523

merged 5 commits into from
Sep 24, 2024

Conversation

WhaleNotPower
Copy link
Contributor

Описание изменений

Теперь более чистая и правильная версия по отношению к прошлой. А именно - подправила преференс сейвфайл, подправила прическам гендеры. Подправила прическам айконстейты и подвигала их в редакторе. (А вообще это является повторением прошлого ПРа)

Почему и что этот ПР улучшит

Данный ПР обновляет количество причесок с 182 общих человеческих, в которых было много дубликатов, до ~100 причесок сейчас. Ставит большинство используемых человеческих причесок в конец DMI файла и в принципе обновляет графически большую часть оставшихся. К всему прочему прически теперь привязаны к гендеру (Кроме унисекс причесок)

Прически которые на данный момент обновлены графически:
image
image
image
image

Авторство

WhalePower - Спрайты и код
Luduk, NinjaPikachu, FatFat - Помощь по коду.

Чеинжлог

🆑

  • image: Обновление большей части причесок и удаление их дубликатов.

@TauKitty
Copy link
Contributor

TauKitty commented Sep 6, 2024

Changelog status: ✔️

@WhaleNotPower
Copy link
Contributor Author

Если что ПР дублирует этот - #13423

"Pompadour" = "Fastline Dandy - Movie",
"Big Pompadour" = "Fastline Dandy - Movie",
"Quiff" = "Quiff - Short",
"Bedhead" = "Bedhead",
Copy link
Contributor

Choose a reason for hiding this comment

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

не хочу пропустить для будущих поколений почему у некоторых причесок есть тег а у этой нету

"Ponytail" = "Ponytail",
"Ponytail 2" = "Ponytail 2",
"Ponytail 3" = "Ponytail 3",
"Side Pony" = "Ponytail (f)",
Copy link
Contributor

Choose a reason for hiding this comment

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

мб лучше Ponytail - Feminine чтобы в стиле того как теги у других причесок

Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield left a comment

Choose a reason for hiding this comment

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

если не учитывать мелочей вроде хорошо, даже совместимость сейвфайлов сделали

@QWxiZ
Copy link

QWxiZ commented Sep 8, 2024

зачем привязывать прически к гендеру
а если я томбойчика хочу отыграть или трапа

@WhaleNotPower
Copy link
Contributor Author

зачем привязывать прически к гендеру а если я томбойчика хочу отыграть или трапа

В обоих гендерах планирую сделать как и женственные так и более мужественные прически. А потому и у гендера MALE, будут всякие femine прически.

@WhaleNotPower
Copy link
Contributor Author

А к всему прочему прически без гендера тоже остаются.

code/modules/client/preferences_savefile.dm Outdated Show resolved Hide resolved
code/modules/client/preferences_savefile.dm Outdated Show resolved Hide resolved
code/modules/client/preferences_savefile.dm Outdated Show resolved Hide resolved
code/modules/client/preferences_savefile.dm Outdated Show resolved Hide resolved
@Danistans2
Copy link
Contributor

Danistans2 commented Sep 10, 2024

К всему прочему прически теперь привязаны к гендеру

Зачем?

Какая невероятная сила мешает женщине носить "мужскую" причёску, а мужчине "женскую"?

@WhaleNotPower WhaleNotPower changed the title New Hair v2.4 (Простите) by WP New Hair v2.4 by WP Sep 10, 2024
@UDaV73rus
Copy link
Member

разделения по полу не должно быть

@WhaleNotPower
Copy link
Contributor Author

К всему прочему прически теперь привязаны к гендеру

Зачем?

Какая невероятная сила мешает женщине носить "мужскую" причёску, а мужчине "женскую"?

Такая, что создаются уникальные прически для полов, которые на налазиют на другую. Если хочешь бегать с мужской прической, берешь мужчину или ищешь унисекс прическу. Что-то я не помню что бы трапы бегали с лысиной на голове, что-то я не помню, что бы трапы бегали с помпадуром.

@WhaleNotPower
Copy link
Contributor Author

разделения по полу не должно быть

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

А если без всего этого. То какой вред в гендерности причесок. Если условные 80 причесок дробятся на 30(female)/30(male)/20(унисекс). У каждого пола по 50 причесок, у каждого пола есть свои уникальные прически.

@Danistans2
Copy link
Contributor

К всему прочему прически теперь привязаны к гендеру

Зачем?

Какая невероятная сила мешает женщине носить "мужскую" причёску, а мужчине "женскую"?

Такая, что создаются уникальные прически для полов, которые на налазиют на другую. Если хочешь бегать с мужской прической, берешь мужчину или ищешь унисекс прическу. Что-то я не помню что бы трапы бегали с лысиной на голове, что-то я не помню, что бы трапы бегали с помпадуром.

Я помню. А даже если и не было — это сомнительная причина отбирать возможность вообще это сделать

@WhaleNotPower
Copy link
Contributor Author

К всему прочему прически теперь привязаны к гендеру

Зачем?
Какая невероятная сила мешает женщине носить "мужскую" причёску, а мужчине "женскую"?

Такая, что создаются уникальные прически для полов, которые на налазиют на другую. Если хочешь бегать с мужской прической, берешь мужчину или ищешь унисекс прическу. Что-то я не помню что бы трапы бегали с лысиной на голове, что-то я не помню, что бы трапы бегали с помпадуром.

Я помню. А даже если и не было — это сомнительная причина отбирать возможность вообще это сделать

Сомнительная идея, это оставлять прически так как есть. Никто не говорит что прически сейчас сделаются и не добавятся новые. В билде еще 20-30 старых причесок по спрайтам, которые я буду переспрайчивать и это только одна из волн обновления причесок. С ними добавляются и новые.

@WhaleNotPower
Copy link
Contributor Author

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

@NinjaPikachuska
Copy link
Contributor

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

@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/obsuzhdenie-pr-a-novye-pricheski-delim-pricheski-na-gendery/43709/1

@LudwigVonChesterfield LudwigVonChesterfield dismissed their stale review September 10, 2024 18:54

тут буча какая-то началась а я не внимательно посмотрел на ВСЕ изменения.

@WhaleNotPower
Copy link
Contributor Author

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

@WhaleNotPower
Copy link
Contributor Author

Жду...я ждун-ждун-ждун.

@WhaleNotPower
Copy link
Contributor Author

@volas прости меня за пинг. Но тут именно твоя проверка нужна, твой апрув на преференс сейвфайл.

@TauKitty TauKitty requested a review from a team September 24, 2024 04:45
@volas volas merged commit 794137e into TauCetiStation:master Sep 24, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants