Удобное взаимодействие с контейнерами by notkripton · Pull Request #9408 · TauCetiStation/TauCetiClassic

notkripton · 2022-06-03T00:34:44Z

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

Актуализация взаимодействий с контейнерами (коробками, рюкзаками) под реалии продвинутых билдов:

dreamseeker.2022-06-03.02-48-10-33.online-video-cutter.com.mp4

А именно:

Добавлен хоткей alt-click по контейнеру для открытия его без поднимания в руки, заменяющий долгое перетаскивание на куклу.
Из контейнеров внутри других контейнеров не вынимая их теперь можно доставать/класть вещи (до третьей вложенности), то есть теперь из спасательной коробки можно вытаскивать штуки прямо на лету, не вынимая ее с рюкзака.

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

Удобство

Авторство

Я, tg

Чеинжлог

🆑

tweak: Добавлен хоткей alt-click по контейнеру для открытия его без поднимания в руки.
tweak[link]: Из контейнеров внутри других контейнеров не вынимая их теперь можно доставать/класть вещи (до третьей вложенности).

TauKitty · 2022-06-03T00:34:46Z

Changelog status: ✔️

Akellazp · 2022-06-03T08:03:38Z

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

хуита, имба

BiQndy · 2022-06-03T08:08:18Z

а что дизлайкают то? Разве этот ПР вносит в игру что то, кроме удобства?

KIBORG04 · 2022-06-03T08:11:07Z

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

Akellazp · 2022-06-03T08:49:29Z

Пушки в коробки положить нельзя

Пушки, которые можно положить в коробки

Mercurialaste · 2022-06-03T08:51:09Z

Пушки в коробки положить нельзя

Пушик, которые можно положить в коробки

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

Sakuya-Izayoi · 2022-06-03T09:41:41Z

code/game/objects/items/weapons/storage/storage.dm

+		if(istype(src, /obj/item/weapon/storage/lockbox))
+			var/obj/item/weapon/storage/lockbox/L = src


внезапно заблокированными могут быть не только локбоксы, но и например коробки с пробирками

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

notkripton · 2022-06-03T09:42:44Z

хуита, имба

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

Sakuya-Izayoi · 2022-06-03T09:44:44Z

Без балдепа не разобраться

TheLone1y · 2022-06-03T10:19:05Z

Втф голос суббалансдепа что-то значит? Ладно как хотите, я считаю что этот механ нам нужен, кто-то может кудахнуть что нельзя вытаскивать предметы из ящика из сумки ведь арря! Но тогда уж удаляйте и механ просмотра, иначе то что персонаж может хоть в 5 раз засунуть что-то в другую вещь и смотреть что там очень не вписывается под аргумент выше, хоткей тоже хороший, недавно новичок в мхелпе спрашивал есть ли хоткей на открытие рюкзака, пришлось ответить нет

simb11 · 2022-06-03T10:32:12Z

Всё ОК и мне нравится, единственное что не нравится, так это то что всякие гении с БС сумками и мешками смогут таскать овердофига лута.

Deahaka · 2022-06-03T10:36:07Z

Кстати, проблемы всяких блюспесс штук не проблемы, если мы наконец пофиксим РнД.

muWander · 2022-06-03T16:26:29Z

ГОСПОДИ СПАСИБО БОЛЬШОЕ НОТКРИПТОН!!!!!

LudwigVonChesterfield

комментарий Гетапа очень валиден

code/game/objects/items/weapons/storage/storage.dm

notkripton · 2022-06-05T14:36:59Z

комментарий Гетапа очень валиден

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

code/game/objects/items/weapons/storage/storage.dm

LudwigVonChesterfield

ето не апрув, ето "не реквест ченджес". мне всё ещё не нравится возможность ковыряться в коробке в коробке

code/game/objects/items/weapons/storage/storage.dm

code/game/objects/items/weapons/storage/visuals.dm

KIBORG04 · 2022-06-09T12:40:48Z

code/game/objects/items/weapons/storage/secure.dm

+/obj/item/weapon/storage/secure/briefcase/try_open(mob/user)
+	if(locked)
+		if(user.in_interaction_vicinity(src))
+			to_chat(user, "<span class='warning'>[src] is locked and cannot be opened!</span>")
+		return FALSE
+	else
+		return ..()


Корректней было бы сделать так без копипасты на проверку висинити.

Suggested change

/obj/item/weapon/storage/secure/briefcase/try_open(mob/user)

if(locked)

if(user.in_interaction_vicinity(src))

to_chat(user, "<span class='warning'>[src] is locked and cannot be opened!</span>")

return FALSE

else

return ..()

/obj/item/weapon/storage/secure/briefcase/try_open(mob/user)

if(locked)

to_chat(user, "<span class='warning'>[src] is locked and cannot be opened!</span>")

return FALSE

return ..()

Хотя если я не прав, то напиши. Хотя вроде все будет работать так же.

Ну и такое же можно сделать с obj/item/weapon/storage/lockbox/try_open(mob/user)

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

Co-authored-by: KIBORG04 <bossmira4@gmail.com>

KIBORG04 · 2022-06-12T13:51:11Z

Без балдепа не разобраться

Балдеп это игроки

Co-authored-by: KIBORG04 <bossmira4@gmail.com>

altclicking

f2b53dd

TauKitty added the Tweak label Jun 3, 2022

Sakuya-Izayoi reviewed Jun 3, 2022

View reviewed changes

LudwigVonChesterfield requested changes Jun 3, 2022

View reviewed changes

code/game/objects/items/weapons/storage/storage.dm Outdated Show resolved Hide resolved

better checks

55f9ecd

notkripton requested review from LudwigVonChesterfield and Sakuya-Izayoi June 4, 2022 13:38

LudwigVonChesterfield reviewed Jun 8, 2022

View reviewed changes

code/game/objects/items/weapons/storage/storage.dm Outdated Show resolved Hide resolved

LudwigVonChesterfield approved these changes Jun 8, 2022

View reviewed changes

in_interaction_vicinity

6ad5edc

KIBORG04 reviewed Jun 9, 2022

View reviewed changes

github-actions bot added the Merge Conflict label Jun 9, 2022

Update code/game/objects/items/weapons/storage/storage.dm

785ea61

Co-authored-by: KIBORG04 <bossmira4@gmail.com>

notkripton requested a review from KIBORG04 June 9, 2022 16:54

Merge branch 'master' into alt-clicking

ed4b46a

github-actions bot removed the Merge Conflict label Jun 12, 2022

KIBORG04 added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jun 12, 2022

Apply suggestions from code review

7f2d80b

Co-authored-by: KIBORG04 <bossmira4@gmail.com>

volas merged commit 8baddfb into TauCetiStation:master Jun 28, 2022

TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Jun 28, 2022

Automatic changelog generation for PR TauCetiStation/TauCetiClassic#9408

1c1f072

notkripton deleted the alt-clicking branch July 9, 2022 17:30

		if(istype(src, /obj/item/weapon/storage/lockbox))
		var/obj/item/weapon/storage/lockbox/L = src

Uh oh!

Conversation

notkripton commented Jun 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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

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

Авторство

Чеинжлог

Uh oh!

TauKitty commented Jun 3, 2022

Changelog status: ✔️

Uh oh!

Akellazp commented Jun 3, 2022

Uh oh!

BiQndy commented Jun 3, 2022

Uh oh!

KIBORG04 commented Jun 3, 2022

Uh oh!

Akellazp commented Jun 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Mercurialaste commented Jun 3, 2022

Uh oh!

Sakuya-Izayoi Jun 3, 2022

Choose a reason for hiding this comment

Uh oh!

notkripton Jun 4, 2022

Choose a reason for hiding this comment

Uh oh!

notkripton commented Jun 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Sakuya-Izayoi commented Jun 3, 2022

Uh oh!

TheLone1y commented Jun 3, 2022

Uh oh!

simb11 commented Jun 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Deahaka commented Jun 3, 2022

Uh oh!

muWander commented Jun 3, 2022

Uh oh!

LudwigVonChesterfield left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

notkripton commented Jun 5, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

LudwigVonChesterfield left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KIBORG04 Jun 9, 2022

Choose a reason for hiding this comment

Uh oh!

KIBORG04 Jun 9, 2022

Choose a reason for hiding this comment

Uh oh!

notkripton Jun 9, 2022

Choose a reason for hiding this comment

Uh oh!

KIBORG04 commented Jun 12, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

notkripton commented Jun 3, 2022 •

edited

Loading

Akellazp commented Jun 3, 2022 •

edited

Loading

notkripton commented Jun 3, 2022 •

edited

Loading

simb11 commented Jun 3, 2022 •

edited

Loading

notkripton commented Jun 5, 2022 •

edited

Loading