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

Ребаланс рюкзаков и сумок #3844

Closed

Conversation

TungstenOctopus
Copy link
Contributor

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

(по сути, порт моего пулреквеста с гаммы)
Сумки теперь вмещают меньше рюкзаков, а рюкзаки нельзя открывать, пока они на спине.

Примерный расклад (фото в цвете)

РЮКЗАК
dreamseeker_ZjPSDL8niP

СУМКА
dreamseeker_8rGvFYo2Cq

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

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

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

Чеинжлог

🆑

  • balance: Сумки теперь вмещают меньше предметов, а рюкзаки нельзя открывать, пока они на спине.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

code/game/objects/items/weapons/storage/backpack.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/storage/backpack.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/storage/storage.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/storage/storage.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/storage/storage.dm Outdated Show resolved Hide resolved
@ViDLiathan
Copy link

Как это коснется Яна? Он не может самостоятельно рюкзак обратно надеть.

@TungstenOctopus
Copy link
Contributor Author

Как это коснется Яна? Он не может самостоятельно рюкзак обратно надеть.

Вот так и коснется

@SpaiR SpaiR added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Aug 24, 2019
@SpaiR
Copy link
Member

SpaiR commented Aug 24, 2019

Собственно, экшн кнопка, которая в левом верхнем углу, твоё ограничение обходит.

@Leshiy-ART
Copy link
Member

Вызывает крайне смешанные чувства. Это было бы не так больно, если бы открытие рюкзака не занимало 3 действия за место одного.
Например автоматически при клике на рюкзак проверять на пустые руки, перемещать рюкзак в активную, менять активную руку и открывать рюкзак. Слить это все в одно действие.
Тогда будет не так больно, и в целом сбалансировано.

@MaximRodionoff
Copy link
Contributor

Портируем фичи с лфвб?

@Artik12344
Copy link

Придётся брать марино-даффлбаги и пуллить их за собой (

@TungstenOctopus
Copy link
Contributor Author

Вызывает крайне смешанные чувства. Это было бы не так больно, если бы открытие рюкзака не занимало 3 действия за место одного.
Например автоматически при клике на рюкзак проверять на пустые руки, перемещать рюкзак в активную, менять активную руку и открывать рюкзак. Слить это все в одно действие.
Тогда будет не так больно, и в целом сбалансировано.

Можно вообще убрать лишние действия и тогда точно будет балансно (нет). Именно в этих трех лишних действиях суть. Что значит автоматически открывать его перемещая в руки? Ты вообще понимаешь, что смысл именно в том, чтобы пользоваться рюкзаком было неудобно, по сравнению с сумкой? Ты сейчас просто предложил полностью обойти изменения ПРа и сделать все так, как было. Это конечно в твоем стиле, но довольно очевидно, что пулреквест делался чтобы кое-что поменять, а не чтоб оставить все как есть.

Copy link
Member

@SpaiR SpaiR left a comment

Choose a reason for hiding this comment

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

Если посмотреть, то у тебя каждый раз перед всеми вызовами open() одни и те же проверки. Как бы намекает, что if (slot_equipped == SLOT_BACK && !reachable_while_equipped) .. else имеет смысл переместить в proc/open.

@Phantom67890
Copy link

Теперь сумки могут вместить 2 магазина для СМГ , дизлайк нафиг это нужно теперь нюкерам надо будет рассчитывать место для магазинов СМГ и всего остального и вообще теперь надо будет как в каком нибудь сурвайвале экономить место думая оставить ли мне аптечку тут или лучше оставить место под 2 сигареты?

@MaximRodionoff
Copy link
Contributor

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

@Vaskozlov
Copy link
Contributor

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

А как быть ОБРовцам? А особенно инженерам или медикам в ОБР?

@Ambassadoing
Copy link
Contributor

Брать снарягу по специальности и не лутать половину оружейки?

@Phantom67890
Copy link

Хрень полная просто зачем всё усложнять когда можно оставить как раньше , теперь прийдеться носить всё в разгрузках и в форме и поясах в итоге прийдеться прокликивать всё чтобы найти одну вещь

@MaximRodionoff
Copy link
Contributor

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

@TauKitty
Copy link
Contributor

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

https://forum.taucetistation.org/t/razdelenie-mehana-sumok-i-ryukzakov/11443/1

@volas volas added the Contentious Спорное изменений требующее обсуждений label Aug 27, 2019
@SpaiR
Copy link
Member

SpaiR commented Aug 31, 2019

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

@SpaiR SpaiR added DO NOT MERGE DNM and removed Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже labels Aug 31, 2019
@SpaiR SpaiR closed this Sep 1, 2019
@TauKitty
Copy link
Contributor

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

https://forum.taucetistation.org/t/golosovanie-nerf-mancha-oficzerov-pr-na-podhode/40231/18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Contentious Спорное изменений требующее обсуждений DO NOT MERGE DNM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet