Fix: Microwave translation

[MrRomainzZ]

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

Переводит название и описание микроволновки

Почему это хорошо для игры

Целостность

Изображения изменений

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

Локалка

Changelog

🆑 Romains
fix: Перевод микроволновки
/:cl:

[AyIong]

Ради микроволновки залезать в кишки TGUI...

[MrRomainzZ]

Ради микроволновки залезать в кишки TGUI...

Тоже в какой-то момент думал, что делаю чушь. Но может быть это ультимативный фикс всего титульника TGUI для будущей локализации?

[AyIong]

Ради микроволновки залезать в кишки TGUI...

Тоже в какой-то момент думал, что делаю чушь. Но может быть это ультимативный фикс всего титульника TGUI для будущей локализации?

Титульник можно вписать отдельно в ТГУИ

[AyIong]

В том же принтере титульник насколько я помню нормальный

[MrRomainzZ]

В том же принтере титульник насколько я помню нормальный

Принтер - фотокопир? Пушто написан на английском

[mattgroy]

Как сайдэффект, многословные титульники будут "Писаться С Прописной Буквы В Каждом Слове", для английского это норма, но в русском это не принято. Если это критично, то в Windows.js можно в

<div className="TitleBar__title">
        {title === title.toLowerCase() ? toTitleCase(title) : title}

сделать что-то типо такой проверки

        {(title === title.toLowerCase() && !hasRussianLetters(title)) ? toTitleCase(title) : capitalizeFirst(title)}

Почему capitalizeFirst(title) вместо title? Чтобы названия по типу "шкафчик с экзоскелетом ГСБ" (то есть в строке есть прописные буквы, но первая - не прописная) так же писались с прописной (прим.: для английского это не так критично, т.к. там имена собственные выводятся в начало, e.g. "HOS's exoskeleton locker").

офк hasRussianLetters(str) и capitalizeFirst(str) нужно будет добавить самому (в string.js есть capitalize(str), но он насильно делает все остальные буквы строчными).

Бонус: если сделать такую проверку, то toTitleCase(str) вообще не придётся менять, т.к. для названий с русскими буквами он применяться не будет.

[MrRomainzZ]

Как сайдэффект, многословные титульники будут "Писаться С Прописной Буквы В Каждом Слове", для английского это норма, но в русском это не принято. Если это критично, то в Windows.js можно в

<div className="TitleBar__title">
        {title === title.toLowerCase() ? toTitleCase(title) : title}

сделать что-то типо такой проверки

        {(title === title.toLowerCase() && !hasRussianLetters(title)) ? toTitleCase(title) : capitalizeFirst(title)}

Почему capitalizeFirst(title) вместо title? Чтобы названия по типу "шкафчик с экзоскелетом ГСБ" (то есть в строке есть прописные буквы, но первая - не прописная) так же писались с прописной (прим.: для английского это не так критично, т.к. там имена собственные выводятся в начало, e.g. "HOS's exoskeleton locker").

офк hasRussianLetters(str) и capitalizeFirst(str) нужно будет добавить самому (в string.js есть capitalize(str), но он насильно делает все остальные буквы строчными).

Бонус: если сделать такую проверку, то toTitleCase(str) вообще не придётся менять, т.к. для названий с русскими буквами он применяться не будет.

Спасибо за наводку. Попробую сделать при возможности

[mattgroy]

Ну или не заморачиваться с TGUI и переводить название в нормальное состояние на бэкенде: tgui_datum.dm

/datum/tgui/New(mob/user, datum/src_object, ui_key, interface, title, width = 0, height = 0, datum/tgui/master_ui = null, datum/ui_state/state = GLOB.default_state)
	...
	if(title)
		src.title = capitalize(sanitize(title))
	...

Минусы: английские многословные титульники НЕ будут "Писаться С Прописной Буквы В Каждом Слове", но кому не похуй?

[MrRomainzZ]

Ну или не заморачиваться с TGUI и переводить название в нормальное состояние на бэкенде: tgui_datum.dm

/datum/tgui/New(mob/user, datum/src_object, ui_key, interface, title, width = 0, height = 0, datum/tgui/master_ui = null, datum/ui_state/state = GLOB.default_state)
	...
	if(title)
		src.title = capitalize(sanitize(title))
	...

Минусы: английские многословные титульники НЕ будут "Писаться С Прописной Буквы В Каждом Слове", но кому не похуй?

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

[mattgroy]

Ну или не заморачиваться с TGUI и переводить название в нормальное состояние на бэкенде: tgui_datum.dm

/datum/tgui/New(mob/user, datum/src_object, ui_key, interface, title, width = 0, height = 0, datum/tgui/master_ui = null, datum/ui_state/state = GLOB.default_state)
	...
	if(title)
		src.title = capitalize(sanitize(title))
	...

Минусы: английские многословные титульники НЕ будут "Писаться С Прописной Буквы В Каждом Слове", но кому не похуй?

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

Ну это я к тому, что "когда" мы все названия переведём на русский, то и минус перестанет быть минусом xdd.

Да и ребята говорят, что мерджить изменения TGUI из оффа проблематично (поправьте если не прав), а так поправил одну строчку на бэке и it just works©. В общем, обсудить бы варианты решения с лидом или мэйнтейнерами репозитория.

[MrRomainzZ]

Благодаря замечанию и наводке от @mattgroy я убрал изменение регулярки и tgui.bundle.js. Теперь заглавная буква появляется везде в титульниках первой, что является:

1. Более ультимативным фиксом
2. Менее ущербным способом

Работает (обозвал микроволновку "микроволновая печь" только в тесте. в файлах она осталась микроволновкой)

Тоже работает, но мы не заметим ибо там выполняется функция из string.js

[MrRomainzZ]

ОБНАРУЖЕНА ОШИБКА. ОТМЕНА МЕРЖА. ТРЕБУЕТСЯ ВРЕМЯ

[MrRomainzZ]

да блять не такое удаление...

[MrRomainzZ]

Это победа

[PhantornRU]

Микроволновка - ок.
Ты уверен что ТГУИшник хочешь в транслейтер закидывать? Закинь в модуль отдельно.

[PhantornRU]

теперь ок

[AyIong]

Это конечно лол