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.

Sign up for GitHub

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

Jump to bottom

Port: keybinds panel #1761

Merged
merged 12 commits into from
Jan 11, 2023
Merged

Port: keybinds panel #1761

merged 12 commits into from
Jan 11, 2023

Conversation

Rerik007
Copy link
Contributor

@Rerik007 Rerik007 commented Dec 19, 2022 •
edited
Loading

Описание

Переносит с оффов к нам этот ПР: ParadiseSS13/Paradise#18166 (их кейбинд панель)
Немного меняет работу SSinput.
Даёт лишь возможность перебиндить. Хоткеи такие же как есть сейчас.
Отключает возможность клиентам без флагов admin и debug пользоваться макросами BYOND, так как всё, для чего легально они использовались можно забиндить в этой панели (эмоции и кастомные эмоции)

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

Авоут: https://discord.com/channels/617003227182792704/755125334097133628/1057603036094152774

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

Снимок экрана 2022-12-19 141233

@definitelynotspaghetti
Copy link
Contributor

pog

@captainnelly
Copy link
Contributor

абилки можно кейбиндить?

@dj-34
Copy link
Contributor

dj-34 commented Dec 19, 2022 •
edited
Loading

ля подгон, проверял кейбайнды на русской раскладке?

@Rerik007
Copy link
Contributor Author

Rerik007 commented Dec 19, 2022 •
edited
Loading

ля подгон, проверял кейбайнды на русской раскладке?

Ага, когда задаётся на английском. И на русском работает. Наоборот нет. Не знаешь ли как решить можно?
В /datum/preferences/proc/capture_keybinding по идее задача идёт

@dj-34
Copy link
Contributor

dj-34 commented Dec 19, 2022 •
edited
Loading

ля подгон, проверял кейбайнды на русской раскладке?

Ага, когда задаётся на английском. И на русском работает. Наоборот нет. Не знаешь ли как решить можно? В /datum/preferences/proc/capture_keybinding по идее задача идёт

Не могу подсказать - не знаю, можешь глянуть здесь, может там че полезного ss220-space/Baystation12#13
Может Валлат чего подскажет

@Rerik007
Copy link
Contributor Author

Rerik007 commented Dec 20, 2022 •
edited
Loading

Будем просто заменять, как сейчас у нас с радиоключами, как я понимаю. Только с русского на английский.
Больше пока не вижу критического.

UPD: Нужно подождать пока зальют ПР с движением.

@Rerik007 Rerik007 marked this pull request as ready for review December 20, 2022 07:36
@Vallat
Copy link
Contributor

Vallat commented Dec 21, 2022

ля подгон, проверял кейбайнды на русской раскладке?

Ага, когда задаётся на английском. И на русском работает. Наоборот нет. Не знаешь ли как решить можно? В /datum/preferences/proc/capture_keybinding по идее задача идёт

Смотри мои ПРы на наш ТГМС, я там чинил это.

@Rerik007
Copy link
Contributor Author

Rerik007 commented Dec 21, 2022 •
edited
Loading

ля подгон, проверял кейбайнды на русской раскладке?

Ага, когда задаётся на английском. И на русском работает. Наоборот нет. Не знаешь ли как решить можно? В /datum/preferences/proc/capture_keybinding по идее задача идёт

Смотри мои ПРы на наш ТГМС, я там чинил это.

thx, ss220-space/TerraGov-Marine-Corps#1
Я взял оттуда.

@Rerik007 Rerik007 marked this pull request as draft December 21, 2022 14:29
@Rerik007 Rerik007 marked this pull request as ready for review December 28, 2022 10:27
@TrashDoxx
Copy link
Contributor

ParadiseSS13/Paradise#20034

@TrashDoxx
Copy link
Contributor

Слыш, программировай дывай.

@Dimach Dimach merged commit 0e95a88 into ss220-space:master220 Jan 11, 2023
@Rerik007 Rerik007 deleted the keybinds-3 branch May 5, 2023 22:28
@Rerik007 Rerik007 restored the keybinds-3 branch May 5, 2023 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Rerik007 @definitelynotspaghetti @captainnelly @dj-34 @Vallat @TrashDoxx @Dimach