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

Adds drask cyrillic accent #1179

Merged
merged 4 commits into from
May 17, 2024

Conversation

HelmsDeepState
Copy link

Что этот PR делает

Теперь автоакцент драсков работает с кириллицей

Тестирование

Поооогоооовооорил на драске

Changelog

🆑
tweak: Теперь у драсков работает автоакцент
/:cl:

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Почему не закинул акцент туда же где и Киданы например, зачем переносить в отдельную папку?

@HelmsDeepState
Copy link
Author

Почему не закинул акцент туда же где и Киданы например, зачем переносить в отдельную папку?

Потому что та папка называется emotes, а эта species

@dj-34
Copy link
Collaborator

dj-34 commented Apr 15, 2024

Так у тебя в Species вон Диона сидит с точно таким же кодом, зачем нужно было вырезать и создавать отдельный файл?

image

@HelmsDeepState
Copy link
Author

HelmsDeepState commented Apr 15, 2024

Так у тебя в Species вон Диона сидит с точно таким же кодом, зачем нужно было вырезать и создавать отдельный файл?

image

Пон. Из модуля emote этот весь файл переместить или пусть там остается?

@dj-34
Copy link
Collaborator

dj-34 commented Apr 15, 2024

Я не понимаю зачем все перемещать в отдельный файл, если это так или иначе связано с эмоцией

HelmsDeepState added 2 commits April 16, 2024 21:33
@HelmsDeepState
Copy link
Author

HelmsDeepState commented Apr 16, 2024

Я не понимаю зачем все перемещать в отдельный файл, если это так или иначе связано с эмоцией

Потому что там в datum species куча барахла, не связанного с эмоцией. Энивей, этот тред стал слишком велик для семи строчек, поэтому я положил всё на место

@HelmsDeepState HelmsDeepState requested a review from dj-34 April 16, 2024 18:46
Comment on lines 74 to 82
autohiss_basic_map = list(
"o" = list ("oo", "ooo"),
"u" = list ("uu", "uuu"),
"о" = list ("оо", "ооо"),
"у" = list ("уу", "ууу")
)
autohiss_extra_map = list(
"m" = list ("mm", "mmm"),
"м" = list ("мм", "ммм")
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts Apr 23, 2024

Choose a reason for hiding this comment

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

А че, одинаковые буквы вроде

Suggested change
autohiss_basic_map = list(
"o" = list ("oo", "ooo"),
"u" = list ("uu", "uuu"),
"о" = list ("оо", "ооо"),
"у" = list ("уу", "ууу")
)
autohiss_extra_map = list(
"m" = list ("mm", "mmm"),
"м" = list ("мм", "ммм")
autohiss_basic_map = list(
"o" = list("oo", "ooo"),
"u" = list("uu", "uuu"),
"о" = list("оо", "ооо"),
"у" = list("уу", "ууу")
)
autohiss_extra_map = list(
"m" = list("mm", "mmm"),
"м" = list("мм", "ммм")

Copy link
Author

Choose a reason for hiding this comment

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

А че, одинаковые буквы вроде

Suggested change
autohiss_basic_map = list(
"o" = list ("oo", "ooo"),
"u" = list ("uu", "uuu"),
"о" = list ("оо", "ооо"),
"у" = list ("уу", "ууу")
)
autohiss_extra_map = list(
"m" = list ("mm", "mmm"),
"м" = list ("мм", "ммм")
autohiss_basic_map = list(
"o" = list("oo", "ooo"),
"u" = list("uu", "uuu"),
"о" = list("оо", "ооо"),
"у" = list("уу", "ууу"),
"m" = list("mm", "mmm"),
"м" = list("мм", "ммм")

Какие одинаковые буквы? Autohiss extra работает если поставить полный акцент, а Autohiss basic, если средний

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ага, вижу, я на примеры смотрел. Тогда пробелы только

Copy link

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label May 15, 2024
@Legendaxe Legendaxe merged commit 34e9f63 into ss220club:master May 17, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 CL валиден Stale ПР долго был не активен, и требует обновления.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants