X Tutup
Skip to content

Fix Clear option not visible when a Resource's @export-ed property is a Script#110189

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
bjornmp:Fix-for-issue-#110178
Sep 19, 2025
Merged

Fix Clear option not visible when a Resource's @export-ed property is a Script#110189
Repiteo merged 1 commit intogodotengine:masterfrom
bjornmp:Fix-for-issue-#110178

Conversation

@bjornmp
Copy link
Contributor

@bjornmp bjornmp commented Sep 2, 2025

Closes #110178

Narrows down the criteria to determine if the property being edited with the EditorResourcePicker is actually the Object->script property.

@bjornmp bjornmp requested a review from a team September 2, 2025 01:37
@AThousandShips AThousandShips added bug topic:editor cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Sep 2, 2025
@AThousandShips AThousandShips added this to the 4.6 milestone Sep 2, 2025
@bjornmp bjornmp force-pushed the Fix-for-issue-#110178 branch from 079fbda to ea4219e Compare September 2, 2025 10:38
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Tested locally, it works as expected. Code looks good to me.

Narrows down the criteria to determine if the property being edited with the `EditorResourcePicker` is actually the `Object->script` property.
@bjornmp bjornmp force-pushed the Fix-for-issue-#110178 branch from ea4219e to 8798387 Compare September 3, 2025 15:39
@Repiteo Repiteo merged commit 5365372 into godotengine:master Sep 19, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 19, 2025

Thanks! Congratulations on your first merged contribution! 🎉

@akien-mga
Copy link
Member

Cherry-picked for 4.5.2.

@akien-mga akien-mga removed cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Jan 16, 2026
@akien-mga akien-mga changed the title Fix Clear option not visible when a Resource's @export-ed property is a Script Fix Clear option not visible when a Resource's @export-ed property is a Script Jan 25, 2026
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.

Inspector: Clear option not visible when a Resource's @export-ed property is a Script

5 participants

X Tutup