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

Квирки на рост #8163

Closed
wants to merge 15 commits into from
Closed

Квирки на рост #8163

wants to merge 15 commits into from

Conversation

FatFatFat1
Copy link
Contributor

@FatFatFat1 FatFatFat1 commented Nov 5, 2021

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

Добавлено 2 Квирка на рост
Добавил флаг для гена гнома ,чтобы он был не такой шакальный

Ген гнома с флагом(теперь):
изображение

Ген гнома без флага(было):
изображение

Куклы с градацией роста:
изображение

Short height и Long height

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

Больше разнообразия среди кукол

Авторство

FatFat

Чеинжлог

🆑 FatFat

  • add: Квирк который увеличивает куклу на ~2 пикселя
  • add: Квирк который уменьшает куклу на ~2 пикселя
  • tweak: Убрал шакальность гена гнома

Добавил 2 квирка на рост
Добавлен флагг к гному,чтобы его не шакалило
@TauKitty
Copy link
Contributor

TauKitty commented Nov 5, 2021

Changelog status: ✔️

@KIBORG04
Copy link
Member

KIBORG04 commented Nov 5, 2021

add: Квирк который увеличивает куклу на ~2 пикселя
add: Квирк который уменьшает куклу на ~2 пикселя
tweak: Флаг KEEP_TOGETHER для гена гнома чтобы ,он был не шакальный

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

P. s.; За антишакаливание отвечает пиксель_скейл, а не кип_тугезер

@FatFatFat1
Copy link
Contributor Author

Да но без этого флага оно шакально, а PIXEL_SCALE ,на человеке и до этого весел

code/modules/mob/living/carbon/human/life.dm Outdated Show resolved Hide resolved
code/game/dna/genes/powers.dm Outdated Show resolved Hide resolved
code/modules/mob/living/carbon/human/life.dm Outdated Show resolved Hide resolved
@UDaV73rus
Copy link
Member

Эти квирки добавились в создание перса?

@FatFatFat1
Copy link
Contributor Author

да

@UDaV73rus
Copy link
Member

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

@FatFatFat1
Copy link
Contributor Author

Я хотел посмотреть пока на такой вариант реализации. Если будет совсем беда,то переделаю/выпилю. Там резайс всего на 5 процентов , он не должен сильно кошмарить спрайты

@Sakuya-Izayoi
Copy link
Contributor

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

@FatFatFat1
Copy link
Contributor Author

FatFatFat1 commented Nov 6, 2021

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

Перенес работу квирков в выдачу квирков
@FatFatFat1
Copy link
Contributor Author

Все сделал

@Lafrien
Copy link

Lafrien commented Nov 6, 2021

А возможность гномам ползать без одежды по вентиляции ты оставил? Квирк на рост гнома даст эту возможность, или он даёт лишь квирк, а не геном? Можно ли поставить квирк и поставить ген гнома, что бы стать еще меньше? Как это будет выглядить?

/datum/dna/gene/basic/midget/activate(mob/living/M, connected, flags)

..(M,connected,flags)

M.pass_flags |= 1

if(ishuman(M))

	var/mob/living/carbon/human/H = M

	H.ventcrawler = 1

	H.update_size_class()

	to_chat(H, "<span class='notice'><b>Ventcrawling allowed</b></span>")`

@FatFatFat1
Copy link
Contributor Author

FatFatFat1 commented Nov 6, 2021

Квирк на низкий рост != Ген гнома.
Если ты взял квирк на низкий/высокий рост ,а потом получил ген гнома , ген гнома перебьет квирк и ты станешь гномом.

Еще раз ,я не делал квирк который дает тебе рост гнома. Есть только низкий и высокий рост. Ген гнома я почти не трогал.

FatFatFat1 and others added 2 commits November 6, 2021 22:58
@KIBORG04 KIBORG04 added the Tweak label Nov 6, 2021
Co-authored-by: KIBORG04 <bossmira4@gmail.com>
@KIBORG04
Copy link
Member

KIBORG04 commented Nov 7, 2021

image

Гномы разве была такими мелкими и со сломанным облачком врайтинга?

@FatFatFat1
Copy link
Contributor Author

Я не трогал значение ресайза гнома. Он как был 0.8 так и остался

@KIBORG04
Copy link
Member

KIBORG04 commented Nov 7, 2021

image
image

Новый гном похож на патау

@KIBORG04
Copy link
Member

KIBORG04 commented Nov 7, 2021

Я не трогал значение ресайза гнома. Он как был 0.8 так и остался

Окошко сломалось из-за кип тугезера, так как начало копировать матрицу у хумана и тоже уменьшаться.

Это без кип_тугезера
image

@KIBORG04 KIBORG04 added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Nov 7, 2021
KIBORG04 and others added 3 commits November 7, 2021 17:07
Теперь не только начальное облачко нересайзится , но и ?/!/...
@FatFatFat1
Copy link
Contributor Author

Пофиксил облачки разговора
изображение

худы теперь не ресайзятся вместе с мобом
@FatFatFat1
Copy link
Contributor Author

Пофиксил худы
изображение

@TauKitty
Copy link
Contributor

TauKitty commented Nov 8, 2021

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/fludilka-chast-2/28659/1556

@SpaiR
Copy link
Member

SpaiR commented Nov 9, 2021

В #8179, по сути, реализована аналогичная фича, но в более широком смысле и чуть интереснее. Я полагаю, что с ней конкретно от данного ПРа останется полезным лишь флаг на незашакаливание при гномировании.

@volas
Copy link
Member

volas commented Nov 9, 2021

тот пр продолжение этого, автор только сменился

@volas volas closed this Nov 9, 2021
@DarthSidiousPalpatine
Copy link
Contributor

Автор не то чтобы сменился, я просто помогаю, а так автор добавился, скорее, чем сменился.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже Tweak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants