Порт системы хоткеев с SS220 Paradise by Bizzonium · Pull Request #2065 · infinitystation/Baystation12

Bizzonium · 2021-07-02T20:59:59Z

Vallat: перенес привычные всем на SS220 хоткеи с парадайза, подогнав под них некоторые существующие. В целом всё должно работать, но в идеале протестить не одним мной, хоть я и пробежался по хуманам/роботам/призракам и вроде бы всё нормально.

Для инфинити скорее всего понадобятся твики биндингов клавиш под привычные, а не под наши.

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

За основу был взят этот PR ParadiseSS13/Paradise#11358, однако без AZERTY раскладки.

Changelog

🆑 Vallat
wip: портирована система хоткеев с SS220 Paradise
/:cl:

* Port hotkey system from paradise * Заменил для некоторых вещей клик СКМ на Альт+СКМ, в целом ни на что не влияет, т.к. у них были аналоги в виде обычного Альт клика

* resolve conflicts * Хоткей для кобуры

* Фиксы хоткеев * Гит ну емаё

Bizzonium · 2021-07-02T21:02:39Z

TODO: пофиксить дёрганное диагональное движение, сменив glide size

quardbreak

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

quardbreak · 2021-07-02T21:04:48Z

code/__defines/colors.dm

 #define COLOR_DARKMODE_DARKBACKGROUND "#171717"
 #define COLOR_DARKMODE_TEXT "#a4bad6"
+
+#define COLOR_INPUT_DISABLED "#F0F0F0"


Цвета и буквы в них с маленькой. У нас стоит ЮнитТест на это.

quardbreak · 2021-07-02T21:05:36Z

code/_helpers/unsorted.dm

 	else
 		step(user.pulling, get_dir(user.pulling.loc, A))
+
+/proc/REF(input)


Заменить на существующий макрос any2ref, если возможно.

Оно в принципе и не нужно как я понял.

quardbreak · 2021-07-02T21:07:15Z

code/game/machinery/Sleeper.dm

+//		if(not_turf_contains_dense_objects(get_turf(get_step(loc, dir))))		//Because of new input system, I don't want to write more cruthes in it
+//			occupant.dropInto(get_step(loc, dir))
+//		else
+		occupant.dropInto(loc)


Чо за костыли?

quardbreak · 2021-07-02T21:07:31Z

code/game/machinery/cryopod.dm

+//	if(not_turf_contains_dense_objects(get_turf(get_step(loc, dir))))		//Because of new input system, I don't want to write more cruthes in it
+//		occupant.dropInto(get_step(loc, dir))
+//	else
+	occupant.dropInto(loc)


Опять же, что за костыли?

quardbreak · 2021-07-02T21:07:37Z

code/game/machinery/cryo.dm

 		occupant.client.eye = occupant.client.mob
 		occupant.client.perspective = MOB_PERSPECTIVE
-	occupant.forceMove(get_step(loc, SOUTH))
+	occupant.forceMove(loc)


IvanDonFon · 2021-07-03T05:54:43Z

Этот ПР добавляет диагональное перемещение?
Это отвратительно

quardbreak · 2021-07-03T07:39:06Z

О, и не маловажное.
Сделайте диагональное движение как опцию в конфиге.
Так можнл сохранить фичу и контролировать её.

quardbreak

Ещё немного ревью.

quardbreak · 2021-07-03T09:30:52Z

code/__defines/misc.dm

+//intent defines
+#define INTENT_HELP   "help"
+#define INTENT_GRAB   "grab"
+#define INTENT_DISARM "disarm"
+#define INTENT_HARM   "harm"


Можно убрать. У нас есть дефайны I_HELP и прочие.

К тому же это не используется нигде, по крайней мере сейчас.

quardbreak · 2021-07-03T09:31:19Z

code/__defines/subsystem-priority.dm

 #define SS_PRIORITY_ICON_UPDATE    20  // Queued icon updates. Mostly used by APCs and tables.

 // Normal
+#define SS_PRIORITY_INPUT		   1000


#define SS_PRIORITY_INPUT 20 по рекомендации Lohikar.

quardbreak · 2021-07-03T09:31:32Z

code/_helpers/unsorted.dm

 	else
 		step(user.pulling, get_dir(user.pulling.loc, A))
+
+/proc/REF(input)


Оно в принципе и не нужно как я понял.

quardbreak · 2021-07-03T09:32:38Z

code/_onclick/click.dm

 		return 1
+	if(modifiers["middle"] && modifiers["alt"])
+		AltMiddleClickOn(A)
+		return 1


Это должно быть выше if(modifiers["middle"]) т.к. выше проеряется голый вариант без модификатора альта и отбрасывает последующие проверки.

github-actions · 2021-07-11T01:12:41Z

Этот PR был неактивен достаточно долго, поэтому он был автоматически помечен как устаревший. Это означает, что существует риск того, что он будет автоматически закрыт через ~ 7 дней. Пожалуйста, устраните все нерешенные вопросы и убедитесь, что ваш PR завершен. Если ваш PR готов, вы должны попросить специалиста по обслуживанию удалить устаревшую метку на вашем PR, чтобы сбросить таймер. Если вы думаете, что за это время ни один сопровождающий не ответит, вы можете закрыть этот PR самостоятельно и переоткрыть его когда будете готовы.

github-actions · 2021-07-19T01:08:19Z

Этот PR был неактивен достаточно долго, поэтому он был автоматически помечен как устаревший. Это означает, что существует риск того, что он будет автоматически закрыт через ~ 7 дней. Пожалуйста, устраните все нерешенные вопросы и убедитесь, что ваш PR завершен. Если ваш PR готов, вы должны попросить специалиста по обслуживанию удалить устаревшую метку на вашем PR, чтобы сбросить таймер. Если вы думаете, что за это время ни один сопровождающий не ответит, вы можете закрыть этот PR самостоятельно и переоткрыть его когда будете готовы.

quardbreak · 2021-07-19T01:25:06Z

ЭЭ бот ты чё офигел.

SidVeld · 2021-07-19T08:44:48Z

@quardbreak

ЭЭ бот ты чё офигел.

github-actions · 2021-08-03T01:12:08Z

Этот PR был неактивен достаточно долго, поэтому он был автоматически помечен как устаревший. Это означает, что существует риск того, что он будет автоматически закрыт через ~ 14 дней. Пожалуйста, устраните все нерешенные вопросы и убедитесь, что ваш PR завершен. Если ваш PR готов, вы должны попросить специалиста по обслуживанию удалить устаревшую метку на вашем PR, чтобы сбросить таймер. Если вы думаете, что за это время ни один сопровождающий не ответит, вы можете закрыть этот PR самостоятельно и переоткрыть его когда будете готовы.

HaCsO · 2022-01-04T10:38:19Z

code/controllers/subsystems/input.dm

+	movement_keys = default_movement_keys.Copy()
+	alt_movement_keys = azerty_movement_keys.Copy()
+
+// Badmins just wanna have fun ♪


ода юникодовские значки в коментариях

HaCsO · 2022-01-04T10:40:21Z

code/game/verbs/ooc.dm

 	set category = "OOC"

+	if(!message)
+		message = input(src.mob, "", "looc \"text\"") as text|null


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

HaCsO · 2022-01-04T10:40:50Z

code/modules/keybindings/bindings_admin.dm

+		if("F5")
+//			if(user.keys_held["Shift"])
+//				user.get_mentor_say()
+//			else


комментарии что бы да?

HaCsO · 2022-01-04T10:43:35Z

code/modules/keybindings/bindings_client.dm

+	//The "tripped" system is to confirm that flooding is still happening after one spike
+	//not entirely sure how byond commands interact in relation to lag
+	//don't want to kick people if a lag spike results in a huge flood of commands being sent
+	if(cache >= MAX_KEYPRESS_AUTOKICK)


уже вижу как за то что люди крутятся на стрелочки их кикают pepega

HaCsO · 2022-01-04T10:44:18Z

code/modules/keybindings/bindings_client.dm

+
+	///Check if the key is short enough to even be a real key
+	if(LAZYLEN(_key) > MAX_KEYPRESS_COMMANDLENGTH)
+		to_chat(src, "<span class='userdanger'>Invalid KeyDown detected! You have been disconnected from the server automatically.</span>")


какой страшный текст, можно использовать SPAN_DANGER("")?

HaCsO · 2022-01-04T10:46:17Z

code/modules/keybindings/bindings_client.dm

+	switch(_key)
+		if("F1")
+			if(keys_held["Ctrl"] && keys_held["Shift"]) // Is this command ever used?
+				winset(src, null, "command=.options")


это что бы что?

HaCsO · 2022-01-04T10:47:41Z

code/modules/keybindings/bindings_human.dm

+					to_chat(user, "<span class='notice'>There's nothing in your backpack to take out.</span>")
+					return
+				var/obj/item/stored = equipped_back.contents[equipped_back.contents.len]
+				if(!stored || stored.on_found(src))


ыыы мусорные комментарии

HaCsO · 2022-01-04T10:50:16Z

code/modules/keybindings/readme.dm

@@ -0,0 +1,42 @@
+# In-code keypress handling system


обязательно было делать этот файл .dm? почему бы не .md или как этот формат у гитовского риадми, ну хотя бы .txt

HaCsO · 2022-01-04T10:53:00Z

ПР настолько старый что логов по юниттестам не осталось плак

IvanDonFon · 2022-01-04T10:53:45Z

Помимо этого, есть информация, что сама система не очень-то рабочая. Большую часть клавиш тупо нельзя переназначить, а некоторые назначения были сломаны, что делает управление очень неудобным. А ещё парадизы - 🤮

Vallat added 3 commits July 2, 2021 23:50

Fix delay (#13) and add holster hotkey (#14)

b0e8257

* resolve conflicts * Хоткей для кобуры

Еще немного фиксов хоткеев (#16)

b100df0

* Фиксы хоткеев * Гит ну емаё

Bizzonium changed the title ~~Port hotkeys from SS220 Paradise~~ Порт системы хоткеев с SS220 Paradise Jul 2, 2021

quardbreak suggested changes Jul 2, 2021

View reviewed changes

quardbreak suggested changes Jul 3, 2021

View reviewed changes

github-actions bot added the Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. label Jul 11, 2021

quardbreak added Unclosable Не может быть автоматически закрыто and removed Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. labels Jul 11, 2021

github-actions bot added the Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. label Jul 19, 2021

quardbreak removed the Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. label Jul 19, 2021

github-actions bot added the Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. label Aug 3, 2021

Archemagus removed the Stale Пуллреквесты и Ишшуи, которые давно не обновлялись и кандидаты на закрытие. label Aug 3, 2021

HaCsO suggested changes Jan 4, 2022

View reviewed changes

Archemagus closed this Jan 7, 2022

Archemagus reopened this Jan 7, 2022

Uh oh!

Comments

Conversation

Bizzonium commented Jul 2, 2021

Changelog

Uh oh!

Bizzonium commented Jul 2, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

quardbreak left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

IvanDonFon commented Jul 3, 2021

Uh oh!

quardbreak commented Jul 3, 2021

Uh oh!

quardbreak left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jul 11, 2021

Uh oh!

github-actions bot commented Jul 19, 2021

Uh oh!

quardbreak commented Jul 19, 2021

Uh oh!

SidVeld commented Jul 19, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Aug 3, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

HaCsO commented Jan 4, 2022

Uh oh!

IvanDonFon commented Jan 4, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Bizzonium commented Jul 2, 2021 •

edited

Loading

SidVeld commented Jul 19, 2021 •

edited

Loading