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

Обновление скилов на установку столов #10900

Merged
merged 2 commits into from
Mar 11, 2023

Conversation

Deahaka
Copy link
Contributor

@Deahaka Deahaka commented Mar 1, 2023

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

Твик #10723, в соответствии с #10890.
Проблема что офицеры могут установить обычные столы в действительности незначительна, так как обычные столы можно опрокидывать и ставить назад без каких либо усилий всем. Также можно легко проползти под столом. Переносим фамблинг на установку стола только таким столам, которые являются серьёзным препятствием.

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

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

Авторство

Чеинжлог

🆑 Deahaka

  • tweak: Установка обычного стола теперь не требует навыков.

@TauKitty
Copy link
Contributor

TauKitty commented Mar 1, 2023

Changelog status: ✔️

@TauKitty TauKitty added the Tweak label Mar 1, 2023
@Deahaka Deahaka changed the title Обновление скилов на установку столов [DNM] Обновление скилов на установку столов Mar 1, 2023
@Deahaka
Copy link
Contributor Author

Deahaka commented Mar 1, 2023

В данный момент это претендует на мерж.

@Deahaka Deahaka changed the title [DNM] Обновление скилов на установку столов Обновление скилов на установку столов Mar 8, 2023
@@ -46,8 +46,6 @@
..()

/obj/item/weapon/table_parts/attack_self(mob/user)
if(!handle_fumbling(user, src, SKILL_TASK_AVERAGE, list(/datum/skill/engineering = SKILL_LEVEL_NOVICE)))
return
var/turf/simulated/T = get_turf(user)
if(!T || !T.CanPass(null, T))
Copy link
Contributor

Choose a reason for hiding this comment

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

было бы хорошо если бы эти проверки были ДО проверки на скилл

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

моё предложение столам дать переменную того как долго их надо ставить.

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

Copy link
Contributor Author

@Deahaka Deahaka Mar 9, 2023

Choose a reason for hiding this comment

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

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

Обычным столам может и нужна задержка, но это касается не только их, а и многих других структур, типо rack'ов. ПР про это могу сделать, если будет ишуй со списком полным где ещё моментальный деплой. Этот ПР про скилы онли.

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.

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

/obj/item/weapon/table_parts
    var/build_time = 0
    
/attack_self()
   if(!checks())
       return
   if(build_time > 0 && !handle_fumbling())
       return
   if(!checks())

@Deahaka Deahaka mentioned this pull request Mar 9, 2023
@KIBORG04 KIBORG04 merged commit daf82fd into TauCetiStation:master Mar 11, 2023
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Mar 11, 2023
@Deahaka Deahaka deleted the skill_table branch March 11, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants