docs(time-range): update article text and preset labels (#DS-4918)#1713
Conversation
…le caption (#DS-4918)
… label (#DS-4918) В третьем примере (time-range-empty-type-list) добавлен кастомный titleTemplate, чтобы подпись выбранного периода шла без предлога «за»/«for» — только в этом примере, глобальное поведение компонента не тронуто.
|
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 |
…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 опция «Все время» теперь идёт последней в списке пресетов — после «Текущий квартал» и «Текущий год».
|
@lskramarov вопрос по объёму локализации. Сейчас обновлены только русский (ru-RU) и английский (en-US). Не тронутыми остались три локали: es-LA, pt-BR, tk-TM. Что именно было изменено для ru-RU и en-US:
Для es-LA / pt-BR / tk-TM всё работает за счёт fallback ( Нужно ли доводить до единообразия и обновить эти три локали (в рамках этого PR или отдельной задачей), или scope осознанно ограничен только ru/en? |
… (#DS-4918) В примере custom-range-types список типов для DI (KBQ_DEFAULT_TIME_RANGE_TYPES) не включал 'range', поэтому компонент считал выбранный произвольный период невалидным при повторном открытии и сбрасывал выбор на первый пресет. Добавлен 'range' в список типов примера — правка только в примере.
Я бы не разбивал на дополнительные задачи, а делал в этой. Давай завтра на синке обсудим. |
There was a problem hiding this comment.
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.
…locales (#DS-4918)
…n templates (#DS-4918)
…lue in examples (#DS-4918)
…stom-option examples (#DS-4918)
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:
*Option), в триггере предлог сохранён; исправлена русская плюральная форма («последние 3 года»); api-репорт core перегенерированtranslationTypeуlast3Years, добавлена защита от незаданного значения вtitleTemplate'range')What should reviewers focus on?
allTimeOption ?? allTimeсохранён для пользовательских конфигов локали.