Skip to content

Move editor UI settings out of project and export data#8609

Open
partyplatter08-lab wants to merge 2 commits into
4ian:masterfrom
partyplatter08-lab:partyplatter08-lab/bounty-1983
Open

Move editor UI settings out of project and export data#8609
partyplatter08-lab wants to merge 2 commits into
4ian:masterfrom
partyplatter08-lab:partyplatter08-lab/bounty-1983

Conversation

@partyplatter08-lab
Copy link
Copy Markdown

Fixes #1983.

Summary

  • Store local editor UI settings in per-project sidecar files under .gdevelop/<project-file>.editor-settings.json instead of local project JSON.
  • Restore sidecar settings when opening local projects and autosaves, while preserving compatibility with legacy embedded settings when no sidecar exists.
  • Strip editor-only layout, external layout, and custom object UI settings from exported runtime project data.

Tests

  • npm test -- --runInBand src/ProjectsStorage/LocalFileStorageProvider/LocalProjectUiSettings.spec.js --watchAll=false
  • npm run flow -- --show-all-errors
  • npx eslint src/ProjectsStorage/LocalFileStorageProvider/LocalProjectUiSettings.js src/ProjectsStorage/LocalFileStorageProvider/LocalProjectUiSettings.spec.js src/ProjectsStorage/LocalFileStorageProvider/LocalProjectWriter.js
  • npx prettier@1.15.3 --list-different src/ProjectsStorage/LocalFileStorageProvider/LocalProjectUiSettings.js src/ProjectsStorage/LocalFileStorageProvider/LocalProjectUiSettings.spec.js src/ProjectsStorage/LocalFileStorageProvider/LocalProjectWriter.js
  • g++ -std=c++17 -DGD_IDE_ONLY -DGD_CORE_API= -I Core -I . -fsyntax-only Core/GDCore/IDE/ProjectStripper.cpp
  • g++ -std=c++17 -DGD_IDE_ONLY -DGD_CORE_API= -I Core -I . -fsyntax-only Core/GDCore/Project/Layout.cpp Core/GDCore/Project/ExternalLayout.cpp Core/GDCore/Project/EventsBasedObjectVariant.cpp

@partyplatter08-lab partyplatter08-lab requested a review from 4ian as a code owner May 17, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[$20] Move UI settings

1 participant