Skip to content

Commit

Permalink
Merge pull request #119 from The1029/master
Browse files Browse the repository at this point in the history
fixed showOngoing toggled on initially when underlying value is falsey.
  • Loading branch information
muness authored Mar 21, 2024
2 parents f8d934e + d5a39ef commit 744f396
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion src/settings/ICSSettingsTab.ts
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,7 @@ class SettingsModal extends Modal {
.setName('Show Ongoing')
.setDesc('Display multi-day events that include target date')
.addToggle(toggle => toggle
.setValue(this.format.showOngoing || true) // Use the new property
.setValue(this.format.showOngoing || false) // Use the new property
.onChange(value => {
this.format.showOngoing = value; // Set the new property
}));
Expand Down

0 comments on commit 744f396

Please sign in to comment.