Skip to content

Commit

Permalink
changed default collapsed state to closed
Browse files Browse the repository at this point in the history
One of the other majestic users commented on the issue asking if the state defaulted
to closed or open. Thinking it through, it should default to closed but the
implementation defaulted to open. So it has now been changed to default to closed
unless there is a failing test. Then it springs open to show the failed test.

The comments were mentioned in Raathigesh#186 but this change is really for Raathigesh#180
  • Loading branch information
Greg Veres committed May 15, 2020
1 parent 08edc68 commit 57f07aa
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ui/test-file/collapseStore.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ export class CollapseStore {
}

static isCollapsed(id: string): boolean {
return CollapseStore.store[id] === true;
let storeState = CollapseStore.store[id];
return storeState === true || storeState === undefined;
}

static setState(id: string, state: boolean) {
Expand Down

0 comments on commit 57f07aa

Please sign in to comment.