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

на ревью #34

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

на ревью #34

wants to merge 17 commits into from

Conversation

maleyvich
Copy link

Описание изменений

Почему и что этот ПР улучшит

Авторство

Чеинжлог

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
else
which_time_is_it = "<b>[mins]</b> minutes."
which_time_is_it = "<b>[mins]</b> минут."
Copy link

Choose a reason for hiding this comment

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

4 минут?

Copy link
Author

Choose a reason for hiding this comment

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

аналогично

else
which_time_is_it = "<b>[hours]</b> hours and [which_time_is_it]"
which_time_is_it = "<b>[hours]</b> часов and [which_time_is_it]"
Copy link

Choose a reason for hiding this comment

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

4 часов?

Copy link
Author

Choose a reason for hiding this comment

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

я забыл че над использовать для времени блииин

return 0
if(SSlag_switch.measures[DISABLE_NON_OBSJOBS])
to_chat(usr, "<span class='notice'>There is an administrative lock on entering the game for non-observers!</span>")
to_chat(usr, "<span class='notice'>Администрация отключила вступление в раунд!</span>")
Copy link
Member

Choose a reason for hiding this comment

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

Неправильный перевод, здесь про то что можно войти только как призрак

Copy link
Author

Choose a reason for hiding this comment

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

войти как призрак не равно вступлению в раунд.

Choose a reason for hiding this comment

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

Лирика думаю тут не особо нужна.

Suggested change
to_chat(usr, "<span class='notice'>Администрация отключила вступление в раунд!</span>")
to_chat(usr, "<span class='notice'>Администрация отключила возможность присоединиться в раунд!</span>")

Хотя ты можешь написать в стиле "Администрация отключила возможность присоединиться в раунд для тех, кто сидит в главном меню!", но это уже выглядит как будто глашатай указ короля издает. Не придумал иного сравнения.

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
else
which_time_is_it = "<b>[mins]</b> minutes."
which_time_is_it = "<b>[mins]</b> минут."
Copy link
Member

Choose a reason for hiding this comment

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

Видимо нам надо добавить инструмент для минут, ибо есть только для секунд

#define PLUR_SECONDS_LEFT(seconds) pluralize_russian(seconds, "секунда", "секунды", "секунд") // "Осталась 1 секунда". Не путайте с нижним.

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved

var/has_quality = client.prefs.selected_quality_name
dat += {"<a id="quality" class="menu_a" href='?src=\ref[src];lobby_be_special=1'>[has_quality ? QUALITY_READY : QUALITY_NOT_READY]</a>"}

dat += {"<a class="menu_a" href='?src=\ref[src];lobby_observe=1'>OBSERVE</a>"}
dat += {"<a class="menu_a" href='?src=\ref[src];lobby_observe=1'>НАБЛЮДЕНИЕ</a>"}

Choose a reason for hiding this comment

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

Наблюдение за чем?

Обычно в играх, особенно это касается соревновательных режимов используется понятие "режим зрителя" (Spectator Mode).

Но для нашей краткости можно использовать следующую комбинацию фраз "стать/быть зрителем" или "в зрители".

Тем более это глагол. Так что для полной краткости можно использовать соответствующее слово "наблюдать"

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
return 0
if(SSlag_switch.measures[DISABLE_NON_OBSJOBS])
to_chat(usr, "<span class='notice'>There is an administrative lock on entering the game for non-observers!</span>")
to_chat(usr, "<span class='notice'>Администрация отключила вступление в раунд!</span>")

Choose a reason for hiding this comment

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

Лирика думаю тут не особо нужна.

Suggested change
to_chat(usr, "<span class='notice'>Администрация отключила вступление в раунд!</span>")
to_chat(usr, "<span class='notice'>Администрация отключила возможность присоединиться в раунд!</span>")

Хотя ты можешь написать в стиле "Администрация отключила возможность присоединиться в раунд для тех, кто сидит в главном меню!", но это уже выглядит как будто глашатай указ короля издает. Не придумал иного сравнения.

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
"Служба Безопасности" = list(jobs = list(), titles = security_positions, color = "#ff9999"),
"Прочие Профессии" = list(jobs = list(), titles = list(), color = "#ffffff", colBreak = TRUE),
"Синтетический Разум" = list(jobs = list(), titles = nonhuman_positions, color = "#ccffcc"),
"Гражданские Отделы" = list(jobs = list(), titles = civilian_positions, color = "#cccccc"),

Choose a reason for hiding this comment

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

Suggested change
"Гражданские Отделы" = list(jobs = list(), titles = civilian_positions, color = "#cccccc"),
"Сервисная Зона" = list(jobs = list(), titles = civilian_positions, color = "#cccccc"),

Во-первых, что значит Гражданский? Сотрудники сервиса Граждане чего? НаноТрейзен? Они не являются гражданами корпорации. Даже если вдруг НаноТрейзен выкупит целую планету, то они будут из-за корпоративного режима называться иначе.

Во-вторых, Сервис, в отличие от остальных, является именно зоной, состоящая из ряда разбросанных отсеков (барбер, голодек, бар, кухня, гидропоника, Его-Величество-Клованство).

В целом можешь почитать определение Сервисной Зоны.

code/modules/mob/dead/new_player/new_player.dm Outdated Show resolved Hide resolved
@@ -408,7 +408,7 @@
var/accurate_length = 600
if(number_of_extra_line_breaks) // We will expand window length for each <br>(Active: [active]) until its reaches 700 (worst cases)
accurate_length = min(700, accurate_length + (number_of_extra_line_breaks * 8))
var/datum/browser/popup = new(src, "latechoices", "Choose Profession", 680, accurate_length)
var/datum/browser/popup = new(src, "latechoices", "Выбор профессии", 680, accurate_length)

Choose a reason for hiding this comment

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

Отличие профессии от должности • Профессия – это род деятельности, а должность – это занимаемое место. Часто должность, которую занимает человек, значительно отличается от приобретенной им профессии. Наличие профессии подтверждает документ о высшем, среднем специальном или профессионально-техническом образовании.

Suggested change
var/datum/browser/popup = new(src, "latechoices", "Выбор профессии", 680, accurate_length)
var/datum/browser/popup = new(src, "latechoices", "Выбор должности", 680, accurate_length)

"Инженерный Отдел" = list(jobs = list(), titles = engineering_positions, color = "#ffd699"),
"Служба Безопасности" = list(jobs = list(), titles = security_positions, color = "#ff9999"),
"Прочие Профессии" = list(jobs = list(), titles = list(), color = "#ffffff", colBreak = TRUE),
"Синтетический Разум" = list(jobs = list(), titles = nonhuman_positions, color = "#ccffcc"),

Choose a reason for hiding this comment

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

Я не знаю как на это реагировать и как это лучше обозвать, но почему-то это словосочетание режет мне глаз. Может @AzzyDreemurr13 или @Pivo24 мне дадут иные идеи?

Самый верный ответ был бы Искусственный Интеллект, ибо он подчеркивает суть наших помощников, но увы и ах - эта конкретная роль станционного ИИ.

Поэтому я предлагаю золотую середину и обозвать их так:

Suggested change
"Синтетический Разум" = list(jobs = list(), titles = nonhuman_positions, color = "#ccffcc"),
"Синтетики" = list(jobs = list(), titles = nonhuman_positions, color = "#ccffcc"),

Выходит лаконично, однако данная часть ревью открыта для обсуждения.

code/modules/mob/dead/new_player/lobby.dm Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants