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

Новые публикации в ленте активности #134

Closed
Takohashi opened this issue Jun 30, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@Takohashi
Copy link

commented Jun 30, 2018

Ключ: newActivityItems

Оригинал:
There {!# [1:is a new item][?:are # new items]} to view. Click here to load {!# [1:it][?:them]}.

Текущий перевод:
{!# [0:Появилось # новых элементов][1:Появился новый элемент][2:Появилось # новых элемента][3:Появилось # новых элемента][4:Появилось # новых элемента][11:Появилось # новых элементов][12:Появилось # новых элементов][13:Появилось # новых элементов][14:Появилось # новых элементов][*1:Появилось # новый элемент][*2:Появилось # новых элемента][*3:Появилось # новых элемента][*4:Появилось # новых элемента][?:Появилось # новых элементов]}. Нажмите здесь для {!# [1:его][*11:Появилось # новых элементов][*12:Появилось # новых элементов][*13:Появилось # новых элементов][*14:Появилось # новых элементов][?:их]} загрузки и просмотра.

Обратил внимание, что неправильно пишется при наличии одного элемента:

2018-06-30 13 52 02

Но посмотрел на перевод и там оказались и другие ошибки:

  1. Это важно! Конструкции вида [*1: ...], [*2: ...] и тд включают в себя [1: ...], [2: ...] и начинают их перекрывать, если указаны в конце, поэтому все конструкции со "*" должны быть указаны в самом начале.
  2. С учётом пункта №1 нет смысла указывать [2:Появилось # новых элемента] при наличии [*2:Появилось # новых элемента] и тп.
  3. Во втором предложении указаны лишние конструкции для 11, 12, 13, 14:
    2018-06-30 15 06 36
  4. Не могу понять, какой смысл в конструкции [0:Появилось # новых элементов]? Без неё всё работает. Обратил внимание, что и в других местах локализации тоже есть конструкции для «0», хотя в оригинале этого нет.

Предлагаю такой перевод (проверил, работает правильно):
{!# [*1:Появился # новый элемент][*2:Появилось # новых элемента][*3:Появилось # новых элемента][*4:Появилось # новых элемента][1:Появился новый элемент][11:Появилось # новых элементов][12:Появилось # новых элементов][13:Появилось # новых элементов][14:Появилось # новых элементов][?:Появилось # новых элементов]}. Нажмите здесь для {!# [1:его][?:их]} загрузки и просмотра.

PS. Если пункт №1 не соблюдён в других аналогичных местах локализации, то там тоже будут выводиться не те конструкции из-за неправильного перекрытия.

@ilyahoilik

This comment has been minimized.

Copy link
Owner

commented Jun 30, 2018

Всегда было наоборот, у первых вариантов был наивысший приоритет. Буду тестировать

@ilyahoilik ilyahoilik added this to the v4.4.0 milestone Jan 29, 2019

@ilyahoilik ilyahoilik modified the milestones: v4.4.0, v4.5.0 Feb 19, 2019

@ilyahoilik ilyahoilik modified the milestones: v4.5.0, v4.4.0 Mar 4, 2019

@ilyahoilik ilyahoilik added the fixed label Mar 4, 2019

@ilyahoilik ilyahoilik closed this in 7e8135b Mar 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.