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

[ru]: sync Doctype #24269

Merged
merged 3 commits into from
Nov 3, 2024
Merged

[ru]: sync Doctype #24269

merged 3 commits into from
Nov 3, 2024

Conversation

sedlukha
Copy link
Contributor

@sedlukha sedlukha commented Oct 28, 2024

Description

Update to the Doctype documentation from the original article https://developer.mozilla.org/en-US/docs/Glossary/Doctype

@sedlukha sedlukha requested a review from a team as a code owner October 28, 2024 14:18
@sedlukha sedlukha requested review from lex111 and removed request for a team October 28, 2024 14:18
@github-actions github-actions bot added the l10n-ru Issues related to Russian content. label Oct 28, 2024
Copy link
Contributor

github-actions bot commented Oct 28, 2024

Preview URLs

Flaws (2)

URL: /ru/docs/Glossary/Doctype
Title: Тип документа
Flaw count: 2

  • broken_links:
    • Can't resolve /ru/docs/Quirks_Mode_and_Standards_Mode
    • Can't resolve /ru/docs/Quirks_Mode_and_Standards_Mode
External URLs (1)

URL: /ru/docs/Glossary/Doctype
Title: Тип документа

(comment last updated: 2024-11-03 20:56:01)

@sedlukha sedlukha changed the title docs: sync Doctype [ru]: sync Doctype Oct 31, 2024
files/ru/glossary/doctype/index.md Outdated Show resolved Hide resolved
@leon-win
Copy link
Member

Привет, @sedlukha !

Благодарю за желание внести вклад в развитие MDN!

Давай ещё немного улучшим перевод этой страницы, раз уже взялись за неё))
Поскольку тут всего два абзаца и три ссылки, то предлагаю полностью синхронизировать её с оригиналом 😎

@@ -1,19 +1,17 @@
---
title: Doctype
slug: Glossary/Doctype
page-type: glossary-definition
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
page-type: glossary-definition

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@leon-win не подскажешь, почему линтер ругается? делал синхронизацию под оригинал, там так же.

Copy link
Member

Choose a reason for hiding this comment

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

page-type должно быть только в оригинальной вресии, для переводов не нужно указывать это свойство.
Логика такая: свойства, которые технически описывают документ, должны быть только в ориганле, а в локалях мы добавляем только slug (выступает идентификатором документа), title (локализованный заголовок документа) и l10n.sourceCommit (указатель на хэш коммита, в котором было последнее изменение оригинала, с которым синхронизирован документ).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

спасибо, @leon-win. я учел эту информацию и поправил другие пулреквесты.

@sedlukha
Copy link
Contributor Author

sedlukha commented Nov 1, 2024

Привет, @sedlukha !

Благодарю за желание внести вклад в развитие MDN!

Давай ещё немного улучшим перевод этой страницы, раз уже взялись за неё)) Поскольку тут всего два абзаца и три ссылки, то предлагаю полностью синхронизировать её с оригиналом 😎

Привет, @leon-win!

Сделал полную синхронизацию

@leon-win
Copy link
Member

leon-win commented Nov 3, 2024

Я немного изменил формулировки (и знаки препинания) + добавил l10n.sourceCommit -- указатель на коммит, в котором было изменение оригнала, с которым мы синхронизируем перевод (про это в комменте выше написал).

Кстати, посмотри, пожалуйста, на документ https://github.com/mdn/translated-content/blob/main/docs/ru/translation-guide.md , там есть основная информация по переводу на русский (например, использование кавычек, тире и т.д.).

@leon-win leon-win merged commit d5ac4a4 into mdn:main Nov 3, 2024
7 checks passed
@sedlukha
Copy link
Contributor Author

sedlukha commented Nov 4, 2024

@leon-win спасибо за ссылку на доку! почему-то не попадалась раньше.

меня вот только немного озадачил тот заголовок, который ты задал для страницы.

Можешь пожалуйста объяснить (на будущее), почему решил изменить заголовок?

Другие переводы страницы, как я вижу, используют "Doctype". Да и нагуглить страницу "doctype" как-будто будет проще, а "тип документа" - это что-то абстрактное.

Также смотрю в доку:

"Используйте терминологию словаря «Веб-стандартов», а также сверяйтесь с глоссарием ниже."

Смотрю словарь:

doctype

докта́йп, конструкция вида в начале документа, которая обозначает используемую версию HTML. Используется браузерами для выбора режима (стандартного, почти стандартного или совместимости).

@leon-win
Copy link
Member

leon-win commented Nov 5, 2024

@sedlukha , привет!

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

В данном случае -- это "тип документа". Разве это не корректный термин?
По поводу абстрактности не совсем понял. Я думаю, что "доктайп" или "doctype" является синонимом "типа документа", слэнговым, профессиональным выражением, допустимым, но не обязательным.

В других переводах мнения тоже разделились, как мы видим, в китайской и корейской локалях тоже решили перевести на свой язык:
https://developer.mozilla.org/ko/docs/Glossary/Doctype
https://developer.mozilla.org/zh-CN/docs/Glossary/Doctype

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

P.S. Кстати я заметил несколько опечаток и неточностей, которые сам и допустил, поэтому создал ещё один PR и исправленим этой страницы #24370.

@sedlukha sedlukha deleted the patch-2 branch November 8, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ru Issues related to Russian content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants