Skip to content

Commit e8cc8cd

Browse files
committed
Fix bug
Properly disable animations when control and stash option is not enabled
1 parent 688582a commit e8cc8cd

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,3 +34,4 @@ yarn-error.log
3434
# emacs backup files
3535

3636
*~
37+
yarn.lock

src/features/envVisualizer/EnvVisualizerAnimation.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { InstrType } from "js-slang/dist/ec-evaluator/types";
22

33
import { AgendaStack, isInstr } from "./compactComponents/AgendaStack";
44
import { AnimationItemComponent } from "./compactComponents/AnimationItemComponent";
5+
import EnvVisualizer from './EnvVisualizer';
56

67
export class EnvVisualizerAnimation {
78
private static animationEnabled = false;
@@ -61,7 +62,8 @@ export class EnvVisualizerAnimation {
6162
}
6263

6364
static startAnimation(): void {
64-
if (!EnvVisualizerAnimation.enableAnimations) {
65+
if (!EnvVisualizerAnimation.animationEnabled || !EnvVisualizer.getAgendaStash()) {
66+
EnvVisualizerAnimation.disableAnimations();
6567
return;
6668
}
6769
EnvVisualizerAnimation.disableAnimations();

src/features/envVisualizer/EnvVisualizerLayout.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -584,6 +584,7 @@ export class Layout {
584584
EnvVisualizer.getAgendaStash() &&
585585
Layout.stashComponent.draw()}
586586
{EnvVisualizer.getCompactLayout() &&
587+
EnvVisualizer.getAgendaStash() &&
587588
EnvVisualizerAnimation.getAnimationComponents().map(c => c.draw())}
588589
</Layer>
589590
</Stage>

0 commit comments

Comments
 (0)