Skip to content

Comments

Change TabContainer's popup button into real Button#114283

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
KoBeWi:adding_physical_buttons
Feb 16, 2026
Merged

Change TabContainer's popup button into real Button#114283
Repiteo merged 1 commit intogodotengine:masterfrom
KoBeWi:adding_physical_buttons

Conversation

@KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Dec 22, 2025

This PR reorganizes TabContainer's internals. The popup button, which used to be drawn manually, is now a real button and is placed inside a new internal HBoxContainer, next to the TabBar. This allowed to simplify the logic and remove some hacks. The functionality/appearance should be more or less the same.

Needed for #113051

@KoBeWi KoBeWi added this to the 4.x milestone Dec 22, 2025
@KoBeWi KoBeWi requested review from a team as code owners December 22, 2025 13:24
@KoBeWi KoBeWi force-pushed the adding_physical_buttons branch from 13c1325 to 4840437 Compare December 22, 2025 23:45
@arkology
Copy link
Contributor

Does it make sense to change the return type of get_internal_container() to just Container?
If the internal container changes in the future (for example, to FlowContainer), this will preserve compatibility.
Or at least into BoxContainer.

@KoBeWi
Copy link
Member Author

KoBeWi commented Dec 24, 2025

The method is not exposed.

@KoBeWi KoBeWi force-pushed the adding_physical_buttons branch from 4840437 to 85283af Compare February 7, 2026 05:10
@KoBeWi KoBeWi requested a review from a team as a code owner February 7, 2026 05:10
@KoBeWi KoBeWi force-pushed the adding_physical_buttons branch from 85283af to c8570b1 Compare February 8, 2026 12:04
YeldhamDev

This comment was marked as resolved.

@KoBeWi KoBeWi force-pushed the adding_physical_buttons branch from c8570b1 to e2cd7f4 Compare February 8, 2026 16:06
@KoBeWi KoBeWi force-pushed the adding_physical_buttons branch from e2cd7f4 to 44f9d7f Compare February 8, 2026 17:13
@KoBeWi KoBeWi requested a review from YeldhamDev February 8, 2026 17:24
Copy link
Member

@YeldhamDev YeldhamDev left a comment

Choose a reason for hiding this comment

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

Everything seems to be in order.

@Repiteo Repiteo modified the milestones: 4.x, 4.7 Feb 10, 2026
@Repiteo Repiteo merged commit cf40653 into godotengine:master Feb 16, 2026
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Feb 16, 2026

Thanks!

@KoBeWi KoBeWi deleted the adding_physical_buttons branch February 16, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants