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

Добавлены заборчики #11329

Conversation

DarthSidiousPalpatine
Copy link
Contributor

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

Добавлен металлический, деревянный и стеклянный заборы. Первые два делаются из металла и дерева, стеклянный не делается ни из чего.
image
image
image
image

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

Заборы для мапперов, заборы для сбух, заборы для ботаников.

Авторство

AndreyGysev и все все все, кто помогал в дискорде.

Чеинжлог

🆑

  • rscadd: Добавлены заборы.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Map Edit Sprites labels Apr 23, 2023
@volas
Copy link
Member

volas commented Apr 23, 2023

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

@DarthSidiousPalpatine
Copy link
Contributor Author

Лижард всегда прав. И Лексакр, несите сюда Вальтеровские заборы, если так уж хотите.

@DarthSidiousPalpatine
Copy link
Contributor Author

Мда, чёрт его знает что не так.

@AgRevol
Copy link
Contributor

AgRevol commented May 16, 2023

Я никто и я апрувлю этот ПР. 🤠
На карте всё адекватно, проблем быть не должно, критичные заборчики убрали.

Непривычно местами? Да. Можно ли это убрать если по факту станет хуже? Абсолютли.

Стоит ли дать волю новому и непривычному чтобы потенциально улучшить стейшон? Я считаю, что да.
Спасибо за внимание

@Deahaka
Copy link
Contributor

Deahaka commented May 16, 2023

Всмысле критичные заборчики убрали? Всё осталось на своих местах с тех пор как ПР появился. Или это мапдиф глючит?

@AgRevol
Copy link
Contributor

AgRevol commented May 16, 2023

Всмысле критичные заборчики убрали? Всё осталось на своих местах с тех пор как ПР появился. Или это мапдиф глючит?

Их было больше, те которые правда были лишними уже убрали. Остальные, - эксперимент

@volas volas requested a review from a team October 24, 2023 06:28
@volas volas marked this pull request as draft October 26, 2023 01:43
Copy link

github-actions bot commented Nov 9, 2023

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Nov 16, 2023
@DarthSidiousPalpatine
Copy link
Contributor Author

Прошу переоткрыть

@Chip11-n Chip11-n reopened this Jan 17, 2024
@DarthSidiousPalpatine
Copy link
Contributor Author

Спасибо, ща допилю

@DarthSidiousPalpatine
Copy link
Contributor Author

Итак, @KIBORG04 исправил твои замечания про экзит и ентер проки, сделал как у столов.
Переделал перекрас перил на image вместо icon.

@volas исправил отображение гейтов на боковых спрайтах, добавив оверлей ниже моба. Работает отлично, хотя и отстаёт на 1-2 тика от основного спрайта((( Но это мелочи.
Также, убрал с голокарт штуки. Сейчас обновляю ветку до мастера, фикшу конфликты и ГОТОВО.

@DarthSidiousPalpatine
Copy link
Contributor Author

ОГО, Никаких конфликтов не появилось за СТОЛЬКО ВРЕМЕНИ. Хотя я скорее новые файлы добавлял чем старые менял. Отлично крч. ГОТОВО!

@DarthSidiousPalpatine DarthSidiousPalpatine marked this pull request as ready for review January 17, 2024 19:30
Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

небольшие замечания

Comment on lines +32 to +33
if(user.is_busy(src))
return FALSE
Copy link
Member

Choose a reason for hiding this comment

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

use_tool уже сам проверяет is_busy, отдельно нет смысла вызывать

Comment on lines +39 to +40
if(user.is_busy(src))
return FALSE
Copy link
Member

Choose a reason for hiding this comment

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

как и тут и ниже

Comment on lines +70 to +71
if(istype(O,/obj/item/projectile))
return TRUE
Copy link
Member

Choose a reason for hiding this comment

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

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

Comment on lines +24 to +28
switch(dir)
if(NORTH)
layer -= 0.01
if(SOUTH)
layer += 0.01
Copy link
Member

Choose a reason for hiding this comment

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

можно было бы вынести в отдельный прок, может быть проверять еще что он прикручен, и нужно добавить это в ротейт прок ниже. И стоит сделать абсолютными значениями вместо инкрементов (типо = INFRONT_MOB_LAYER + 0.01), сейчас не очень понятно и с поворотами будет еще страннее.

set category = "Object"
set src in oview(1)

if(isobserver(usr)) //to stop ghosts from rotating
Copy link
Member

Choose a reason for hiding this comment

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

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

Copy link
Contributor

Choose a reason for hiding this comment

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

incapacitated() вероятно


/obj/structure/fence/metal/proc/change_color(new_color)
cut_overlay(Rail)
Rail = image(icon, "[icon_state]_color")
Copy link
Member

Choose a reason for hiding this comment

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

стоит использовать mutable_appearance - это как image, только не отправляется клиенту пока ты с ним работаешь. Чуть оптимальней, и с ними как-то меньше багов если использовать оверлеем

return ..()

/obj/structure/fence/metal/deconstruct(disassembled)
new /obj/item/stack/sheet/metal(loc, 2)
Copy link
Member

Choose a reason for hiding this comment

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

NODECONSTRUCT

cancover = FALSE

/obj/structure/fence/glass/deconstruct(disassembled)
new /obj/item/stack/sheet/glass(loc, 2)
Copy link
Member

Choose a reason for hiding this comment

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

NODECONSTRUCT

var/id
var/open = FALSE

var/image/under_sprite
Copy link
Member

Choose a reason for hiding this comment

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

аналогично с mutable_appearance

Comment on lines +269 to +270
if(istype(mover,/obj/item/projectile))
return TRUE
Copy link
Member

Choose a reason for hiding this comment

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

аналогично как выше про прожектайлы

@DarthSidiousPalpatine
Copy link
Contributor Author

Ок, всё решу.

@volas volas mentioned this pull request Jan 29, 2024
Copy link
Contributor

@Deahaka Deahaka left a comment

Choose a reason for hiding this comment

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

Изменение хорошее, очевидных недочётов не заметил

Copy link

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Feb 21, 2024
@DarthSidiousPalpatine
Copy link
Contributor Author

Спасибо воласу за счастливое детство.

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.

None yet

9 participants