Skip to content

Корректирует получение номера версии - #1382

Merged
solarrust merged 1 commit into
doka-guide:mainfrom
vitya-ne:fix-baseline-support-checking
Sep 28, 2026
Merged

solarrust merged 1 commit into
doka-guide:mainfrom
vitya-ne:fix-baseline-support-checking

Conversation

@vitya-ne

@vitya-ne vitya-ne commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Исправляет получение номера версии браузера из данных web-features

Что сделано

Как оказалось, кроме числа, строка support может содержать знак ≤.
Например: некоторые свойства table:

изображение

Текущая база содержит более 200 '≤' в значениях версий.
Несмотря на знак, значение '≤' для версии, означает что фича точно поддерживается от указанной версии и выше.
См. issue : web-platform-dx/web-features#1018 (comment)

В текущем коде знак ≤ препятствует получению номера версии из строки, так как условие:

!Number.isNaN(item[key])

где item[key], это например '≤4' не будет выполняться.

Предлагаемое изменение удаляет знак ≤ перед попыткой преобразования в число.

Как проверить

Дока для проверки: colgroup
В текущей версии неверно отображается поддержка свойства в Safari

Чеклист

  • npm run check проходит
  • Для изменений во внешнем виде приложены скриншоты — до и после
  • Есть тест: на исправленный баг — воспроизводящий его, на новую функциональность — покрывающий её
  • Если тест написать было не на что (шаблон, CSS, вёрстка) — выше написано, чем проверяли вместо него
  • Соседние тесты просмотрены и поправлены, если проверяли устаревшее поведение
  • Документация обновлена, если поменялись команды, запуск или устройство сборки

Превью контента из efa2e2c опубликовано.

@solarrust solarrust left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Спасибо!

@solarrust
solarrust merged commit 10fd308 into doka-guide:main Sep 28, 2026
7 checks passed
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.

2 participants