X Tutup
Skip to content

Improve style changed signal in EditorDock#113690

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
KoBeWi:take_signal_away_from_user
Dec 15, 2025
Merged

Improve style changed signal in EditorDock#113690
Repiteo merged 1 commit intogodotengine:masterfrom
KoBeWi:take_signal_away_from_user

Conversation

@KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Dec 6, 2025

Changes tab_style_changed signal from user signal to properly registered internal signal. Also added _emit_changed() method that emits the signal using SNAME.

@KoBeWi KoBeWi added this to the 4.x milestone Dec 6, 2025
@KoBeWi KoBeWi requested a review from a team December 6, 2025 18:52
@KoBeWi KoBeWi force-pushed the take_signal_away_from_user branch 2 times, most recently from 64f9210 to 158752e Compare December 11, 2025 15:06
Copy link
Contributor

@kitbdev kitbdev left a comment

Choose a reason for hiding this comment

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

Makes sense

@@ -93,6 +97,7 @@ void EditorDock::_bind_methods() {
ADD_PROPERTY(PropertyInfo(Variant::INT, "available_layouts", PROPERTY_HINT_FLAGS, "Vertical:1,Horizontal:2,Floating:3"), "set_available_layouts", "get_available_layouts");
Copy link
Contributor

Choose a reason for hiding this comment

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

Unrelated but this should be 1 2 4 since they are flags

@KoBeWi KoBeWi force-pushed the take_signal_away_from_user branch from 158752e to 0ed400a Compare December 13, 2025 20:06
@Repiteo Repiteo modified the milestones: 4.x, 4.6 Dec 13, 2025
@Repiteo Repiteo merged commit 00c5af7 into godotengine:master Dec 15, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Dec 15, 2025

Thanks!

@KoBeWi KoBeWi deleted the take_signal_away_from_user branch December 15, 2025 14:18
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.

3 participants

X Tutup