Real stealth for ninja and syndie box #1075
Conversation
There was a problem hiding this comment.
Хорошее изменение, но есть нюансы:
- Если мы делаем измерения за пределами наших папок и пространства имён, то мы обязаны комментариями помечать таковые изменения, например:
//SS220-something
DoSomething();
или
//SS220-something begin
DoSomething();
DoStuff();
//SS220-something end
- Изменения переменной в StealthComponent не самое лучшее решение, т. к. это стандартное значение датафилда. Лучше поменять именно это поле в нужном прототипе.
|
Усиление невидимости одобряли? |
|
Отоброжение оверлея худов при этом скрывается? |
Добавил описание внесённых изменений с тегом SS220.
Добавил описание вносимых изменений.
Правильно? |
| @@ -74,6 +74,11 @@ | |||
| parent: MobHuman | |||
| id: MobHumanSpaceNinja | |||
| components: | |||
| /// <summary> | |||
There was a problem hiding this comment.
в прототипах комменатрии ставятся через #, т.е.:
#SS220-some-change
- type: Comp
| @@ -57,6 +57,7 @@ public sealed partial class StealthComponent : Component | |||
|
|
|||
| /// <summary> | |||
| /// Minimum visibility. Note that the visual effect caps out at -1, but this value is allowed to be larger or smaller. | |||
| /// SS220 - changed maximum visibility to be equal to minimum so that ninja will be actually invisible, not just blur effect everyone sees immediately. | |||
There was a problem hiding this comment.
как я написал в пункте 2 ревью, изменения в коде лучше в принципе убрать.
попробуй поменять компонент StealthClothing в одежде ниндзя
… код, на который ссылаются другие системы
Через гит пуш неактуальный комментарий не захотел удаляться, удалил так.
github-actions
bot
added
the
Changes: Prototypes
|
Убрал изменения в коде, всё сделано через прототипы. Базовые системы, на которые что-либо ссылается не затронуты. |
|
@Surani1 мнение |
Тема со стеллсом - прямое изменение баланса, это наболевшая проблема как у игроков, не любящих когда их ВХшут, так и у администрации. Нужно создать трекер с предложением и обязательно голосованием, в котором можно будет понять, каково мнение игроков на этот счёт. После этого я пингану адуча и там уже будет решение, про два пункта выше так-же жду ответа. |
Я против трекера, потому что там 95% васяны которые спамят пиво на всё подряд и им в принципе посрать. С момента когда я начал админить с печальной регулярностью этот вопрос всплывал в спорах. Я считаю лучшее решение - сделать так, чтобы игрок не думал "а не нарушаю ли я правила?" во время просто игры. Плюс тот инвиз что есть вовсе не инвиз, толку от него минимум, только атмосферу рушит ибо каждый думает что не должен его видеть, хотя отчётливо видит. По-моему данное изменение решит как и вопрос споров о правилах, так и добавит комфорта игрокам, так и подходит по смыслу для ниндзи, который должен быть всё-таки скрытный. |
С трекером решать @Ady4ik ( ну и в общем посмотри подумай ) Проблема с коробкой - из-за твоих изменений она стала какой-то имбалансной, скорость захода в инвиз слишком быстрая и при передвижении медленно спадает. Был момент где я пытался исправить нормально инвиз, но по итогу вышла сильная каша, поэтому идею отложил на будущее. |
|
|
Всем селом думали как это сделать, по итогу придумали: теперь ниндзя красиво модно уходит в инвиз плавно. Content.client.2024.06.07.-.23.54.10.01.mp4 |
removed bricked line
Fixed action PhaseCloack
|
@Ady4ik мнение после изменений? |
Описание PR
Изменил прозрачность эффекта стелса, теперь его действительно трудно заметить.
Медиа
Content.client.2024.06.01.-.17.58.25.09.mp4
Проверки
Изменения
🆑 Lancevrot