Skip to content

Add Rename and Set as start scene to scene context menus #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JamieW105 merged 3 commits into from
Oct 21, 2026
Merged

Add Rename and Set as start scene to scene context menus #46

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
31bea16
feat(editor): rename scenes and choose a persistent start scene
jamietwells10-pixel Oct 21, 2026
c4ed919
docs: record start scene configuration in project architecture
jamietwells10-pixel Oct 21, 2026
fd58e1c
fix(editor): preserve external change tracking for inactive scene act…
jamietwells10-pixel Oct 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
363 changes: 336 additions & 27 deletions Engine/apps/editor/src/main.rs
View file
Open in desktop

Large diffs are not rendered by default.

20 changes: 9 additions & 11 deletions Engine/crates/engine-editor/src/document.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ use std::collections::{BTreeMap, BTreeSet};
use std::path::{Path, PathBuf};
use thiserror::Error;

mod scene_files;

#[derive(Debug, Error)]
pub enum AuthoringError {
#[error(transparent)]
Expand All @@ -35,6 +37,8 @@ pub enum AuthoringError {
},
#[error("scene path must stay inside the project scenes folder")]
UnsafeScenePath,
#[error("scene file operation failed: {0}")]
SceneFileOperation(String),
}

/// One source scene and its centralized, checkpoint-aware edit history.
Expand All @@ -60,22 +64,16 @@ pub struct AuthoringDocument {
}

impl AuthoringDocument {
/// Opens the project's main scene, including backup recovery when required.
/// Opens the project's start scene, including backup recovery when required.
///
/// # Errors
///
/// Returns an error when project paths, scene recovery, migration, or world construction fails.
pub fn open(project: Project, read_only: bool) -> Result<Self, AuthoringError> {
let scene_directory = project.directory(VirtualDirectory::Scenes)?;
// New projects use scene/{name}.scene. Keep opening the original
// scenes/main.rscene location so existing projects need no migration.
let preferred = scene_directory.join("main.scene");
let legacy = scene_directory.join("main.rscene");
let scene_path = if preferred.is_file() || !legacy.is_file() {
preferred
} else {
legacy
};
if let Some(relative) = project.descriptor().start_scene.clone() {
return Self::open_scene(project, relative, read_only);
}
let scene_path = project.start_scene_path()?;
let (document, recovery_read_only, recovered_from_backup) = if scene_path.is_file() {
let loaded = load_scene_recovering(&scene_path)?;
(
Expand Down
Loading
Loading