Skip to content

docs(time-range): update article text and preset labels (#DS-4918)#1713

Merged
rmnturov merged 14 commits into
mainfrom
docs/DS-4918-Update-time-range-article-text
Jul 10, 2026
Merged

docs(time-range): update article text and preset labels (#DS-4918)#1713
rmnturov merged 14 commits into
mainfrom
docs/DS-4918-Update-time-range-article-text

Conversation

@rmnturov

@rmnturov rmnturov commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

По итогам вычитки статьи Time Range обновлены текст статьи (RU/EN), локализация пресетов и примеры документации; учтены замечания из ревью.

Макет: https://www.figma.com/design/HxMaLxGvkFuopugeFHNqVJ/koobiq-%C2%B7-guides?node-id=40002393-59163&t=fTXnCLk89T54qdBV-4

List of notable changes:

  • обновлён текст статьи (RU/EN); пример min-max приведён в соответствие с текстом — ограничение дат с 01.01.2015 по 31.12.2017
  • обновлена локализация пресетов во всех пяти локалях (ru, en, es-LA, pt-BR, tk-TM): подписи в списке — с заглавной буквы и без предлога (новые ключи *Option), в триггере предлог сохранён; исправлена русская плюральная форма («последние 3 года»); api-репорт core перегенерирован
  • обновлены примеры: локализованные подписи пресетов и плейсхолдеры («Период»/“Period”), порядок пресетов в overview, исправлен translationType у last3Years, добавлена защита от незаданного значения в titleTemplate
  • исправлен сброс выбранного произвольного периода при повторном открытии попапа (в списке типов примера custom-range-types не было 'range')

What should reviewers focus on?

  • Переводы es-LA / pt-BR / tk-TM — сделаны по английскому образцу; fallback allTimeOption ?? allTime сохранён для пользовательских конфигов локали.

rmnturov added 3 commits July 8, 2026 14:00
… label (#DS-4918)

В третьем примере (time-range-empty-type-list) добавлен кастомный titleTemplate, чтобы
подпись выбранного периода шла без предлога «за»/«for» — только в этом примере,
глобальное поведение компонента не тронуто.
@rmnturov rmnturov self-assigned this Jul 8, 2026
@rmnturov rmnturov requested review from NikGurev and lskramarov July 8, 2026 12:30
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 6b00356):

https://koobiq-next--prs-1713-fv4aacgt.web.app

(expires Sun, 12 Jul 2026 11:27:18 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

rmnturov added 4 commits July 8, 2026 19:57
…s (#DS-4918)

Убрана подпись о диапазоне под примером. Текст незаполненного поля изменён на
«Выбрать период»/«Select period». Для произвольного периода убран предлог
«за»/«for», а для остальных значений первая буква теперь заглавная.
Правки только в этом примере, остальные примеры не затронуты.
Подписи выбранных значений в примере теперь локализованы: без предлога
«за»/«for» и с заглавной буквы (например, «Последние 3 минуты»). Текст
незаполненного поля изменён на «Период»/«Period». Для произвольного периода
подпись показывает не только даты, но и время.
Правки только в этом примере, остальные примеры и компонент не затронуты.
…DS-4918)

CI-проверка check-api падала: enUSLocaleData в core.api.md не содержал новые
allTimeOption/currentQuarterOption/currentYearOption поля, добавленные ранее
в en-US.ts. Файл сгенерирован заново через approve-api.
… (#DS-4918)

В примере overview опция «Все время» теперь идёт последней в списке
пресетов — после «Текущий квартал» и «Текущий год».
@rmnturov

rmnturov commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

@lskramarov вопрос по объёму локализации.

Сейчас обновлены только русский (ru-RU) и английский (en-US). Не тронутыми остались три локали: es-LA, pt-BR, tk-TM.

Что именно было изменено для ru-RU и en-US:

  • добавлены ключи allTimeOption / currentQuarterOption / currentYearOption — подписи опций в списке пресетов (с заглавной буквы и без предлога «за»/«for»), в отличие от подписи в триггере, где предлог сохранён;
  • в durationTemplate.option первая буква сделана заглавной («Последние …» / «Last …») — только для списка опций в поповере, шаблон триггера (title) не меняли.

Для es-LA / pt-BR / tk-TM всё работает за счёт fallback (allTimeOption ?? allTime), но там в списке пресетов останутся старые подписи — с предлогом и со строчной буквы, то есть визуально разъедутся с ru/en.

Нужно ли доводить до единообразия и обновить эти три локали (в рамках этого PR или отдельной задачей), или scope осознанно ограничен только ru/en?

… (#DS-4918)

В примере custom-range-types список типов для DI (KBQ_DEFAULT_TIME_RANGE_TYPES)
не включал 'range', поэтому компонент считал выбранный произвольный период
невалидным при повторном открытии и сбрасывал выбор на первый пресет.
Добавлен 'range' в список типов примера — правка только в примере.
@rmnturov rmnturov marked this pull request as ready for review July 8, 2026 15:48
@rmnturov rmnturov added documentation Improvements or additions to documentation help wanted Extra attention is needed labels Jul 8, 2026
@lskramarov

Copy link
Copy Markdown
Contributor

@lskramarov вопрос по объёму локализации.

Нужно ли доводить до единообразия и обновить эти три локали (в рамках этого PR или отдельной задачей), или scope осознанно ограничен только ru/en?

Я бы не разбивал на дополнительные задачи, а делал в этой. Давай завтра на синке обсудим.

Comment thread packages/components/time-range/time-range.ru.md
Comment thread packages/components/core/locales/types.ts

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Time Range documentation article (RU/EN), refines locale strings for preset labels (including separate “option” labels for the presets list), and adjusts multiple docs examples to better match the updated UX and localization rules.

Changes:

  • Added separate locale labels for presets list entries (*Option) and updated the time-range editor template to use them with fallbacks.
  • Updated RU/EN locale strings for time range presets and duration option formatting (capitalization / wording).
  • Updated docs examples (overview/min-max/empty-type-list/custom-range-types) to reflect revised labels, placeholders, ordering, and custom trigger/title rendering.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tools/public_api_guard/components/core.api.md Updates public API snapshot to include new optional locale fields for time-range preset option labels.
packages/docs-examples/components/time-range/time-range-overview/time-range-overview-example.ts Adds localized “Events/События” caption and reorders presets list (moves allTime after quarter/year).
packages/docs-examples/components/time-range/time-range-min-max/time-range-min-max-example.ts Reworks the example trigger rendering via titleTemplate (placeholder + range formatting + capitalization) and adds link/icon UI.
packages/docs-examples/components/time-range/time-range-empty-type-list/time-range-empty-type-list-example.ts Adds titleTemplate to render the custom range title without the “for/за” prefix.
packages/docs-examples/components/time-range/time-range-custom-range-types/time-range-custom-range-types-example.ts Localizes preset labels via DateFormatter.duration, adds localized placeholder, and includes 'range' in default types to prevent selection reset.
packages/components/time-range/time-range.ru.md Updates RU article text and section descriptions for Time Range.
packages/components/time-range/time-range.en.md Updates EN article text and section descriptions for Time Range.
packages/components/time-range/time-range-editor.html Uses new *Option locale fields (with fallback to existing labels) for the presets list labels.
packages/components/time-range/snapshots/time-range.spec.ts.snap Updates snapshots to reflect new RU labels (“Период”, “Текущий год”, etc.).
packages/components/core/locales/types.ts Extends KbqTimeRangeLocaleConfig with optional allTimeOption/currentQuarterOption/currentYearOption fields (+ docs).
packages/components/core/locales/ru-RU.ts Updates RU time-range labels and adds new *Option translations; capitalizes duration option strings.
packages/components/core/locales/en-US.ts Updates EN time-range labels and adds new *Option translations; capitalizes duration option strings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/components/time-range/time-range.ru.md
Comment thread packages/components/time-range/time-range.en.md
@rmnturov rmnturov requested a review from NikGurev July 9, 2026 11:45
@rmnturov rmnturov requested a review from artembelik July 9, 2026 11:45
@rmnturov rmnturov merged commit 290e6a2 into main Jul 10, 2026
10 checks passed
@rmnturov rmnturov deleted the docs/DS-4918-Update-time-range-article-text branch July 10, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants