Skip to content
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.

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

Editor: Store background in IndexedDB #22023

Merged
merged 4 commits into from
Jun 19, 2021
Merged

Editor: Store background in IndexedDB #22023

merged 4 commits into from
Jun 19, 2021

Conversation

mrdoob
Copy link
Owner

@mrdoob mrdoob commented Jun 19, 2021

Related issue: #18479 #16154

Description

Store the background texture (and equirectangular texture) on IndexedDB.

@mrdoob mrdoob added this to the r130 milestone Jun 19, 2021
@@ -131,8 +131,7 @@ Editor.prototype = {
this.scene.uuid = scene.uuid;
this.scene.name = scene.name;

this.scene.background = ( scene.background !== null ) ? scene.background.clone() : null;
Copy link
Owner Author

Choose a reason for hiding this comment

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

I can't remember why we were cloning scene.background.

Copy link
Collaborator

@Mugen87 Mugen87 Jun 19, 2021

Choose a reason for hiding this comment

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

Maybe because scene.fog is cloned, too^^?

It seems the line was added like that right from the beginning: 750973b#diff-c7a0a223e819b2c4e36644546146290d9d14f241cf78daa5438faf335a2aff35R124

Copy link
Collaborator

Choose a reason for hiding this comment

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

Since you have changed the code, it makes sense to align:

if ( scene.fog !== null ) this.scene.fog = scene.fog.clone();

Copy link
Owner Author

Choose a reason for hiding this comment

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

Done! 64c6990

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.

2 participants