Skip to content

Escape GaeaEditorSettings calls during runtime#577

Merged
Zehir merged 4 commits intogaea-godot:2.0from
Zehir:escape_settings
Feb 15, 2026
Merged

Escape GaeaEditorSettings calls during runtime#577
Zehir merged 4 commits intogaea-godot:2.0from
Zehir:escape_settings

Conversation

@Zehir
Copy link
Collaborator

@Zehir Zehir commented Feb 10, 2026

This PR is an other part for #573

The idea is to remove the reference of the GaeaEditorSettings class by loading directly it when needed.

  • Bonus optimization on the way

@Zehir Zehir added optimization organization Movement of code around, refactoring, removal of code. labels Feb 10, 2026
@Zehir Zehir requested review from BenjaTK and cullumi and removed request for BenjaTK February 10, 2026 21:54
Copy link
Collaborator

@BenjaTK BenjaTK left a comment

Choose a reason for hiding this comment

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

LGTM

@Zehir Zehir merged commit 59c78ae into gaea-godot:2.0 Feb 15, 2026
5 checks passed
@Zehir Zehir deleted the escape_settings branch February 15, 2026 21:23
@BenjaTK BenjaTK added 🐞 bug Something isn't working topic: editor Changes to the UI or UX of the editor. and removed organization Movement of code around, refactoring, removal of code. labels Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working optimization topic: editor Changes to the UI or UX of the editor.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants