Skip to content

feat!: remove AdaptiveTabs - #411

Open
amje wants to merge 1 commit into
mainfrom
remove-tabs
Open

amje wants to merge 1 commit into
mainfrom
remove-tabs

Conversation

@amje

@amje amje commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@amje
amje requested review from ValeraS and korvin89 as code owners October 16, 2026 10:51
@gravity-ui-bot

Copy link
Copy Markdown
Contributor

Preview is ready.

@korvin89 korvin89 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.

🤖 AI generated

This review was done with AI assistance. I validated every comment myself, but the wording — including the points I found myself — was written by the AI.

Severity legend

  • 🔴 blocker — should be fixed before merge.
  • 🟠 should-fix — a real problem; worth fixing before or right after merge. If you can't get to it now, please open an issue and leave a comment linking it.
  • 🔵 nit — minor; author's discretion.
  • ❓ question — I need more context; not necessarily a defect.

Findings outside the diff

  • README.md:34 — 🟠 should-fix — The removal of AdaptiveTabs is a breaking change, but the migration guide does not mention it: this PR only drops the existing AdaptiveTabs bullet from the "Migrating to UIKit 8" list. Consumers whose import {AdaptiveTabs} from '@gravity-ui/components' breaks after the upgrade will not find the reason or the migration path in the README. Please add a bullet to this list stating that AdaptiveTabs is removed and what to use instead.

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.

3 participants