Code improvements after 4.1.0 release (#5022)

* Align spacing

* Fix typo

* Improve readability

* Update lockfile

In 30e762e the peer dependency was upgraded, but the lock file was not included in that commit.

Author: Frank3K

package-lock.json

@@ -32,8 +32,8 @@ interface PlayAnimationOptions {
 
 interface AppendAnimationOptions {
   pingpong: boolean, repetitions: number|null, weight: number,
-      timeScale: number, fade: boolean|number, warp: boolean|number,
-      relativeWarp: boolean, time: number|null
+  timeScale: number, fade: boolean|number, warp: boolean|number,
+  relativeWarp: boolean, time: number|null
 }
 
 interface DetachAnimationOptions {
@@ -115,9 +115,9 @@ export const AnimationMixin = <T extends Constructor<ModelViewerElementBase>>(
         if (!this[$scene].appendedAnimations.includes(e.action._clip.name)) {
           this[$paused] = true;
         } else {
-          const filterdList = this[$scene].appendedAnimations.filter(
+          const filteredList = this[$scene].appendedAnimations.filter(
               i => i !== e.action._clip.name);
-          this[$scene].appendedAnimations = filterdList;
+          this[$scene].appendedAnimations = filteredList;
         }
         this.dispatchEvent(new CustomEvent('finished'));
       });

packages/model-viewer/src/features/animation.ts

@@ -89,14 +89,16 @@ export const EnvironmentMixin = <T extends Constructor<ModelViewerElementBase>>(
       }
 
       if (changedProperties.has('toneMapping')) {
-        this[$scene].toneMapping = this.toneMapping === 'aces' ?
-            ACESFilmicToneMapping :
-            this.toneMapping === 'agx'      ? AgXToneMapping :
-            this.toneMapping === 'reinhard' ? ReinhardToneMapping :
-            this.toneMapping === 'cineon'   ? CineonToneMapping :
-            this.toneMapping === 'linear'   ? LinearToneMapping :
-            this.toneMapping === 'none'     ? NoToneMapping :
-                                              NeutralToneMapping;
+        const TONE_MAPPING = new Map([
+          ['aces', ACESFilmicToneMapping],
+          ['agx', AgXToneMapping],
+          ['reinhard', ReinhardToneMapping],
+          ['cineon', CineonToneMapping],
+          ['linear', LinearToneMapping],
+          ['none', NoToneMapping]
+        ]);
+        
+        this[$scene].toneMapping = TONE_MAPPING.get(this.toneMapping) ?? NeutralToneMapping;
         this[$needsRender]();
       }