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 TTS name examine #2535

Closed
wants to merge 4 commits into from
Closed

Conversation

lzk228
Copy link
Collaborator

@lzk228 lzk228 commented Aug 29, 2024

добавил кнопочку для осмотра названия ттс
кнопочку не видно если у персонажа нет ттс или скрыта личность(убрал т.к. очень спорно)
работает с голосовой маской

2024-08-29_13-50-27.mp4

🆑

  • add: При осмотре персонажа можно будет увидеть название его голоса ТТС.

@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation and removed Status: Needs Review This PR requires new reviews before it can be merged labels Aug 29, 2024
@NotSoDana
Copy link
Contributor

Ахуеть, почему это только щас добавили

@lzk228
Copy link
Collaborator Author

lzk228 commented Aug 29, 2024

потому что всем похуй было, а я чисто от нехуй делать

@NotSoDana
Copy link
Contributor

Нихуя себе "нехуй делать". Был бы я женщиной да я бы тебя всю ночь ублажала

@Meguneri
Copy link
Contributor

Реал круто

@YuiFann
Copy link

YuiFann commented Aug 29, 2024

Нихуя себе "нехуй делать". Был бы я женщиной да я бы тебя всю ночь ублажала

для такого женщиной быть не обязательно

@Morb0
Copy link
Member

Morb0 commented Aug 31, 2024

Нельзя ли метаебить с этим? Чекая изменился ли голос или подобное
И отбели иконку

@lzk228
Copy link
Collaborator Author

lzk228 commented Aug 31, 2024

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

И отбели иконку

уже отбелил в апстриме

@NotSoDana
Copy link
Contributor

Нельзя ли метаебить с этим? Чекая изменился ли голос или подобное И отбели иконку

Это ведь как раз и противодействует метаёбу голосовой маски. Ибо невзирая на то что ты поставил имя человека, тебе не поверят из-за другого ТТС. Теперь получится сразу узнавать что ставить хотя бы.

@Morb0
Copy link
Member

Morb0 commented Aug 31, 2024

По хорошему если это нужно только для войс маски, дать просто ей фичу по клику на человека или каким-то другим способом копировать его установленный голос в маску

@lzk228
Copy link
Collaborator Author

lzk228 commented Aug 31, 2024

ну это не только для маски, а в целом для "ого у него крутой голос! интересно как он называется"

Copy link
Contributor

@pxc1984 pxc1984 left a comment

Choose a reason for hiding this comment

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

seems good

using Robust.Shared.Prototypes;
using Robust.Shared.Utility;

namespace Content.Server.Corvax.TTS
Copy link
Collaborator

Choose a reason for hiding this comment

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

File-scoped namespace

@lzk228
Copy link
Collaborator Author

lzk228 commented Oct 30, 2024

пр is derelict так что стоит сюда имплементировать систему из DetailedExamine после решения issue space-wizards/space-station-14#31689

@lzk228 lzk228 closed this Oct 30, 2024
@lzk228 lzk228 added the Status: Frozen This PR can't be merged, because of a current freeze label Oct 30, 2024
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 Status: Frozen This PR can't be merged, because of a current freeze Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants