Skip to content

Added several properties to tab panel#2640

Merged
Corepex merged 3 commits into
1.xfrom
add-properties-to-tab-panel
Dec 2, 2025
Merged

Added several properties to tab panel#2640
Corepex merged 3 commits into
1.xfrom
add-properties-to-tab-panel

Conversation

@Corepex

@Corepex Corepex commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

Changes in this pull request

Related to https://github.com/pimcore/portal-engine/pull/820

Additional info

This pull request improves the flexibility of the Tabpanel component by adding support for controlled and uncontrolled tab selection. The main changes are the addition of defaultActiveKey, activeKey, and onChange props to Tabpanel, allowing parent components to manage which tab is active or respond to tab changes. The implementation ensures these props are correctly passed down to the underlying tab system, while also cleaning up unused props in the Tabs component.

Enhancements to tab selection and control:

  • Added defaultActiveKey, activeKey, and onChange props to the TabpanelProps interface and passed them through the Tabpanel component, enabling both controlled and uncontrolled tab usage. [1] [2] [3]
  • Removed the unused onChange prop from the Tabs component implementation, ensuring that tab change handling is now managed at the Tabpanel level. [1] [2]

@Corepex Corepex self-assigned this Dec 1, 2025
@Corepex
Corepex requested a review from vin0401 December 1, 2025 16:50
@sonarqubecloud

sonarqubecloud Bot commented Dec 1, 2025

Copy link
Copy Markdown

@Corepex
Corepex merged commit 66fbd1a into 1.x Dec 2, 2025
1 check passed
@Corepex
Corepex deleted the add-properties-to-tab-panel branch December 2, 2025 09:12
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 2, 2025
@markus-moser markus-moser added this to the next milestone Dec 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants