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

add: Security tactical flashlight #5948

Merged
merged 5 commits into from
Nov 11, 2024

Conversation

PlayerUnknown14
Copy link
Member

@PlayerUnknown14 PlayerUnknown14 commented Sep 26, 2024

Описание

Добавлен "Security tactical flashlight" - тактический фонарик для СБ, надевается в слот уха, можно взять в СБ вендоре.

Показатели идентичны обычному фонарику, способности тенелингов и прочее исправно работают.

Ссылка на предложение/Причина создания ПР

https://discord.com/channels/617003227182792704/755125334097133628/1263219513286070324

Демонстрация изменений

image

Тесты

Смотрел спрайты на локалке, юзал на фонарь абилку теней, покупал фонарь в SecTech'е.

@github-actions github-actions bot added the DMI Affects DMI files label Sep 26, 2024
@Ssheya
Copy link
Contributor

Ssheya commented Sep 26, 2024

честно с учётом сколько у нас сб, и что всего один секмат, я бы увеличил их количество, вещь крутая

@PlayerUnknown14
Copy link
Member Author

честно с учётом сколько у нас сб, и что всего один секмат, я бы увеличил их количество, вещь крутая

Если под секматом ты имел ввиду автомат с экипировкой СБ, то их в бриге 2. 4 фонаря на отдел - достаточно (так решил Кисик).

NightDawnFox
NightDawnFox previously approved these changes Sep 27, 2024
@NightDawnFox NightDawnFox added the Has Approve Have been approved by somebody from Review Team and awaiting for one more approve label Sep 27, 2024
Copy link
Member

@Daeberdir Daeberdir left a comment

Choose a reason for hiding this comment

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

Попробуй сделать наложение оверлея а-ля "света" на слое шапки.

@NightDawnFox
Copy link
Contributor

Да зачем, это же с нуля спрайты рисовать, как минимум

@PlayerUnknown14
Copy link
Member Author

Попробуй сделать наложение оверлея а-ля "света" на слое шапки.

Для этого спрайты нужно трогать или я что-то не понимаю?

@Daeberdir
Copy link
Member

Да зачем, это же с нуля спрайты рисовать, как минимум

Мне бы не хотелось, как игроку, видеть в раунде исполнение без нормального оверлея света. Если хотите вводить так - то бе меня.

@PlayerUnknown14
Copy link
Member Author

Ревью бы...

@Den109G
Copy link
Contributor

Den109G commented Oct 16, 2024

change request реши.

@PlayerUnknown14
Copy link
Member Author

change request реши.

Скорее всего я туплю, но я не смог найти, как принять/отказаться от изменений. Всё перерыл, нужную кнопку не нашёл.

@Den109G
Copy link
Contributor

Den109G commented Oct 16, 2024

Сделай то, что написано в Change Request. Или скажи, почему нельзя такое сделать. Пока это не решено, мы не можем замержить этот ПР.

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Oct 27, 2024
@PlayerUnknown14
Copy link
Member Author

d

Отказано, т.к. Сплит не хочет перерисовывать спрайты, оставляю текущие.

@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Nov 11, 2024
@Vladisvell Vladisvell merged commit 64a8a16 into ss220-space:master220 Nov 11, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files Has Approve Have been approved by somebody from Review Team and awaiting for one more approve
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants