Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added ability to have Tabs in TabContainer at bottom #44420

Closed

Conversation

EricEzaM
Copy link
Contributor

@EricEzaM EricEzaM commented Dec 16, 2020

Exposed this functionality to GD Script (as expected).
Images for the "bottom" tabs were added to the default theme. The editor theme was also changed to have the option of using bottom-aligned tabs. The textures added are just the existing textures flipped upside down.
Also simplified some logic to remove unnecessary duplication of code and nested if/else statements.

bY7KGEmHUb.mp4

Closes godotengine/godot-proposals#1986

@EricEzaM EricEzaM requested a review from a team as a code owner December 16, 2020 07:18
@EricEzaM
Copy link
Contributor Author

Could potentially add the option to toggle this for the editor tab containers, in the popup with changing dock positions and "make floating".

Exposed this functionality to GD Script (as expected).
Some cleanup was also done with the default and editor themes.
Also simplified some logic to remove unneccesary duplication of code and nested if/else statements.
@EricEzaM EricEzaM force-pushed the PR/tab-container-tabs-at-bottom branch from 6cb6ce5 to e9ff3b9 Compare October 1, 2021 13:58
@EricEzaM EricEzaM requested a review from a team as a code owner October 1, 2021 13:58
@EricEzaM
Copy link
Contributor Author

EricEzaM commented Oct 1, 2021

Rebased. Updated video in OP

@YuriSizov
Copy link
Contributor

I noticed that in master we have *_mirrored styles for RTL on buttons. So I guess the ship has sailed on keeping it simple. So if explicit bottom styles are easier to implement, then I guess it's okay to move forward with it. I can always add a button to mirror a stylebox to the theme editor, or something like that.

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.

Add left, right and bottom location options for TabContainer
6 participants