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

Fix small typos in l10n CSS #542

Merged
merged 3 commits into from
Feb 3, 2022
Merged

Fix small typos in l10n CSS #542

merged 3 commits into from
Feb 3, 2022

Conversation

cw118
Copy link
Member

@cw118 cw118 commented Jan 13, 2022

Fixes some typos for Russian (closes mdn/translated-content#3602) and small mistakes for French

@myshov
Copy link

myshov commented Jan 16, 2022

👍 for Russian translation

@mdn/yari-content-fr could you review French translations?

Copy link
Member

@SphinxKnight SphinxKnight left a comment

Choose a reason for hiding this comment

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

👍 for @mdn/yari-content-fr as well, thanks for the fix @cw118 and for the ping @myshov :)

@tanjaslo
Copy link

👍 for Russian translation
https://developer.mozilla.org/ru/docs/Web/CSS/grid по этой ссылке нужно ещё
поправить "соответвующему" -> "соответствующему"

@myshov
Copy link

myshov commented Jan 18, 2022

@tanjaslo I don't have rights to add changes to this pr. I think we can resolve the issues related to https://developer.mozilla.org/ru/docs/Web/CSS/grid in separate pr

@cw118
Copy link
Member Author

cw118 commented Jan 18, 2022

I can make the changes a little later when I have access to a proper device—might as well fix as many typos as possible here?

@myshov
Copy link

myshov commented Jan 18, 2022

@cw118 great! There are two typos here:

  • (соответвует should be соответствует)
    "ru": "<a href=\"/ru/docs/Web/CSS/box-shadow#Interpolation\" title=\"Цвет, абсцисса, ордината, размытие и распространение (если применено) списка теней интерполируются независимо. Если внутреннее значение какой-либо пары теней различается в обоих списках, интерполизуется весь список. Если один список меньше остальных, он дополняется прозрачностью теней со всей их длинной установленной в 0, а его внутреннее значение соответвует длинному списку.\">список теней</a>"
  • (соответвующему should be соответствующему)
    "ru": "относятся к соответвующему размеру области содержимого"

@cw118
Copy link
Member Author

cw118 commented Jan 18, 2022

@myshov I've fixed both in my most recent commit, please feel free to let me know if there's anything else.
И спасибо @tanjaslo, что указали на ошибки ))

@myshov
Copy link

myshov commented Jan 19, 2022

👍for new Russian translations

@Rumyra
Copy link
Contributor

Rumyra commented Jan 31, 2022

Let me know if this is good to go and I'll merge 👍

Copy link
Member

@SphinxKnight SphinxKnight left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @cw118 !

@cw118
Copy link
Member Author

cw118 commented Feb 3, 2022

@myshov @sashasushko if possible, could someone from mdn/yari-content-ru please confirm the Russian corrections are good to go?

@cw118
Copy link
Member Author

cw118 commented Feb 3, 2022

Hi @Rumyra, I hope it's alright to ping you — I believe this PR is ready to merge now :)

@Rumyra
Copy link
Contributor

Rumyra commented Feb 3, 2022

@cw118 yes - and thanks :) The package won't publish immediately so there might be a delay until you see the changes just fyi 👍

@Rumyra Rumyra merged commit d3fb416 into mdn:main Feb 3, 2022
@cw118 cw118 deleted the l10n-typos branch February 3, 2022 15:48
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.

Issue with "padding": (short summary here please)
6 participants