Skip to content

Conversation

@lodetrick
Copy link
Contributor

Closes #112611

Previously, if the editor theme was changed, the bottom panel's background would not be updated. This is because when the bottom panel retrieved the necessary stylebox it retrieved the overridden previous theme rather than the new theme. Removing the override is a simple method to force the retrieved stylebox to be correct.

I'm not too sure why it retrieved the override in the first place because overrides are specific to nodes but this method worked well.

@lodetrick lodetrick requested review from a team as code owners November 10, 2025 23:52
@ryevdokimov
Copy link
Contributor

Not sure how this should be addressed, but possibly related issue and a comment I made there: #112549 (comment)

bottom_panel->_theme_changed();

Convention says this should be a private method, and shouldn't the bottom panel handle theme changes itself via the notification system?

_theme_changed();

Having that method called on repaint (tab changed) as well seems strange. Hard to tell what is going on with a quick glance.

@KoBeWi
Copy link
Member

KoBeWi commented Nov 11, 2025

Alternate fix is getting the theme stylebox either directly from theme (EditorNode::get_editor_theme()->get_stylebox()) or from another control (EditorNode::get_gui_base()->get_theme_stylebox()).

I'm not too sure why it retrieved the override in the first place because overrides are specific to nodes

The override came from BottomPanel theme variation.

possibly related issue and a comment I made there

It's not.

@lodetrick lodetrick force-pushed the fix-bottom-panel-theming branch from 409764b to 3f5339a Compare November 13, 2025 00:46
@lodetrick
Copy link
Contributor Author

I ended up using the alternate solution you mentioned, it solved the issue nicely.

@Repiteo Repiteo merged commit 45b427f into godotengine:master Nov 14, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 14, 2025

Thanks!

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.

Bottom panel theming is broken

5 participants