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

Russian translation: improvements of output-theming #16282

Merged
merged 2 commits into from May 15, 2018

Conversation

lex111
Copy link
Contributor

@lex111 lex111 commented May 14, 2018

Q A
Is bugfix? yes (doc)
New feature? no
Breaks BC? no
Tests pass? --
Fixed issues --

@SilverFire SilverFire added the type:docs Documentation label May 15, 2018
@lex111
Copy link
Contributor Author

lex111 commented May 15, 2018

How about merge?

@@ -9,7 +9,7 @@
заменяются файлы отображений. Главным образом, стоит настроить следующие свойства [[yii\base\Theme]]:

- [[yii\base\Theme::basePath]]: базовая директория, в которой размещены темизированные ресурсы (CSS, JS, изображения,
и так далее).
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
Contributor Author

Choose a reason for hiding this comment

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

Просто сокращение, так лучше ведь?

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
Contributor Author

Choose a reason for hiding this comment

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

Нужно тогда памятку (гайдлан) создать, я ещё некоторое время хочу позаниматься русской версией документации.

Copy link
Member

Choose a reason for hiding this comment

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

Памятка есть: https://github.com/yiisoft/yii2/blob/master/docs/internals-ru/translation-workflow.md. Если дополните, будет замечательно. Спасибо!

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
Contributor Author

@lex111 lex111 May 16, 2018

Choose a reason for hiding this comment

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

Неплохо бы создать, это поспособствует росту контрибьюторов, если, конечно, это вас интересует.
По поводу документации - я бы ещё использовал терминологию, которая используется в официальном русском переводе, например, закрытые методы, а не приватные и т.д.
Кстати, а почему вы не вынесли документацию в отдельный репозиторий, было ведь намного удобнее?

Copy link
Member

Choose a reason for hiding this comment

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

В отдельный не выносили потому как так удобней рассматривать pull request — разом и фичи и доки.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Но не гибко, всё в одном месте, огромный список PR, сложно ориентироваться.

Copy link
Member

@samdark samdark May 16, 2018

Choose a reason for hiding this comment

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

По доке практически нет. Мы их мёржим, чаще всего, быстро.

@samdark samdark merged commit 39548bf into yiisoft:master May 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:docs Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants