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

Hidden description subdermal implants fix #2787

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Alwayswannahunt
Copy link

Описание PR

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

Почему / Баланс

Руки никак не доходили, вот дошли.

Технические детали

Выделен публичный метод для hidden description, который просто пушит информацию из компонента.
Добавлен компонент который при осмотре ентити пытается найти в контейнерах ентити с hidden description и пушит их описание.

Медиа

Учёным:
Осмотрел имплантер из аплинка, вколол в медика
Медиком-предателем:
Осмотрел обычный имплантер из медфаба, достал им из себя имплант-аплинк, осмотрел его.

Снимок экрана 2024-11-19 175028
Снимок экрана 2024-11-19 175040
Снимок экрана 2024-11-19 173504
Снимок экрана 2024-11-19 175221

Требования

Список изменений

🆑 AlwyAnri

  • fix: Исправлено отображение скрытого описания имплантера! Теперь оно зависит от импланта внутри имплантера!

@github-actions github-actions bot added size/L Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation S: Needs Review S: Untriaged labels Nov 19, 2024
@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

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

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

@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

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

@Vonsant
Copy link
Collaborator

Vonsant commented Nov 19, 2024

@lzk228 сишарп сам смотри, я тупае.

@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

не буду ревьювить код того, что мне не нравится 😎💪

@Vonsant
Copy link
Collaborator

Vonsant commented Nov 19, 2024

Вообще в идеале на это надо посмотреть автору системы.

@Vonsant
Copy link
Collaborator

Vonsant commented Nov 19, 2024

Скину ему, пусть посмотрит.

@Alwayswannahunt
Copy link
Author

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

Если это просьба вернуть описание/название имплантеров, которые inject-only, окей верну.

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

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

@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

Если это просьба вернуть описание/название имплантеров, которые inject-only, окей верну.

это реально вопрос был

@Alwayswannahunt
Copy link
Author

Если это просьба вернуть описание/название имплантеров, которые inject-only, окей верну.

это реально вопрос был

От этого ответ не сильно поменяется - я просто скопировал .ftl файлы, до удаления описания с имплантеров, поэтому мне без разницы, если считаете, что нужно название (что логично), то верну в текущее состояние. Моя цель была убрать проблему, что все в итоге зовут учёных смотреть, а те не могут ничего сказать, потому что описание не на импланте, а на имплантере

@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

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

image

@Alwayswannahunt
Copy link
Author

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

image

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

@lzk228
Copy link
Collaborator

lzk228 commented Nov 19, 2024

2024-11-19_19-29-56.mp4

@Alwayswannahunt
Copy link
Author

2024-11-19_19-29-56.mp4

Всё, понял, все знают все импланты, тогда вопросов нету. Сори, энциклопедию имплантов вселенной не читал.
Если будет желание вернутся - пингуйте.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation S: Needs Review S: Untriaged size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants