Skip to content

docs: add TabControl migration guide - #401

Closed
NotYoojun with Copilot wants to merge 2 commits into
tabcontrol-evtsfrom
copilot/sub-pr-400
Closed

NotYoojun with Copilot wants to merge 2 commits into
tabcontrol-evtsfrom
copilot/sub-pr-400

Conversation

Copilot AI commented Nov 18, 2025

Copy link
Copy Markdown
Contributor

Added comprehensive migration guide for the TabControl refactoring that changed event naming and close button visibility control.

Changes

  • Migration guide: Created docs/TabControl-Migration-Guide.md documenting breaking changes and upgrade path
    • Event system: TabItemClosingTabCloseRequested, new TabItemHelper.CloseRequested
    • Close button visibility: IsAddTabButtonVisible → per-tab IsClosable property
    • Manual tab removal now required (no automatic removal after event)
  • Code examples: Before/after patterns for XAML, code-behind, and MVVM scenarios
  • Migration checklist: Quick reference for developers upgrading

Example

Before:

<TabControl ui:TabControlHelper.TabItemClosing="OnClosing" 
            ui:TabControlHelper.IsAddTabButtonVisible="False">

After:

<TabControl ui:TabControlHelper.TabCloseRequested="OnCloseRequested">
    <TabItem Header="Closable" />
    <TabItem Header="Fixed" ui:TabItemHelper.IsClosable="False" />
</TabControl>
private void OnCloseRequested(object sender, TabViewTabCloseRequestedEventArgs e)
{
    // Manual removal now required
    ((TabControl)sender).Items.Remove(e.Tab);
}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: NotYoojun <54567489+NotYoojun@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor tab close button and handling logic docs: add TabControl migration guide Nov 18, 2025
Copilot AI requested a review from NotYoojun November 18, 2025 09:01
@NotYoojun NotYoojun closed this Nov 18, 2025
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