From 366133318501d73da0ac097dacbe66067aa0ea13 Mon Sep 17 00:00:00 2001
From: bpnguyen107 <105088397+bpnguyen107@users.noreply.github.com>
Date: Thu, 29 Aug 2024 05:11:56 -0700
Subject: [PATCH] Move 'ignore reviews before' to advanced (#3381)

* moved ignore setting to advanced

* Update CONTRIBUTORS

* Match width of other text inputs (dae)

The width was inconsistent before as well, but moving it next to
text inputs made it more obvious.
---
 CONTRIBUTORS                                  |  1 +
 ts/routes/deck-options/AdvancedOptions.svelte | 19 +++++++++++++++++++
 ts/routes/deck-options/DateInput.svelte       |  6 +++---
 ts/routes/deck-options/FsrsOptions.svelte     |  6 ------
 .../deck-options/FsrsOptionsOuter.svelte      |  5 -----
 5 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index 64c831d14bc..b0bb6b97dd0 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -189,6 +189,7 @@ Dillon Baldwin <https://github.com/DillBal>
 Voczi <https://github.com/voczi>
 Ben Nguyen <105088397+bpnguyen107@users.noreply.github.com>   
 Themis Demetriades <themis100@outlook.com>
+
 ********************
 
 The text of the 3 clause BSD license follows:
diff --git a/ts/routes/deck-options/AdvancedOptions.svelte b/ts/routes/deck-options/AdvancedOptions.svelte
index 48d486b3818..2bc95668cbe 100644
--- a/ts/routes/deck-options/AdvancedOptions.svelte
+++ b/ts/routes/deck-options/AdvancedOptions.svelte
@@ -19,6 +19,7 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
     import type { DeckOptionsState } from "./lib";
     import SpinBoxFloatRow from "./SpinBoxFloatRow.svelte";
     import SpinBoxRow from "./SpinBoxRow.svelte";
+    import DateInput from "./DateInput.svelte";
 
     export let state: DeckOptionsState;
     export let api: Record<string, never>;
@@ -39,6 +40,11 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
             help: tr.deckConfigHistoricalRetentionTooltip(),
             sched: HelpItemScheduler.FSRS,
         },
+        ignoreRevlogsBeforeMs: {
+            title: tr.deckConfigIgnoreBefore(),
+            help: tr.deckConfigIgnoreBeforeTooltip2(),
+            sched: HelpItemScheduler.FSRS,
+        },
         startingEase: {
             title: tr.schedulingStartingEase(),
             help: tr.deckConfigStartingEaseTooltip(),
@@ -216,6 +222,19 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
                     {tr.deckConfigHistoricalRetention()}
                 </SettingTitle>
             </SpinBoxFloatRow>
+
+            <Item>
+                <DateInput bind:date={$config.ignoreRevlogsBeforeDate}>
+                    <SettingTitle
+                        on:click={() =>
+                            openHelpModal(
+                                Object.keys(settings).indexOf("ignoreRevlogsBeforeMs"),
+                            )}
+                    >
+                        {tr.deckConfigIgnoreBefore()}
+                    </SettingTitle>
+                </DateInput>
+            </Item>
         {/if}
 
         <Item>
diff --git a/ts/routes/deck-options/DateInput.svelte b/ts/routes/deck-options/DateInput.svelte
index 9af3dd9f0c4..a3cba4f5315 100644
--- a/ts/routes/deck-options/DateInput.svelte
+++ b/ts/routes/deck-options/DateInput.svelte
@@ -13,11 +13,11 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
 <div>
     <ConfigInput>
-        <Row --cols={2}>
-            <Col>
+        <Row --cols={13}>
+            <Col --col-size={7} breakpoint="xs">
                 <slot />
             </Col>
-            <Col>
+            <Col --col-size={6} breakpoint="xs">
                 <input bind:value={date} type="date" />
             </Col>
         </Row>
diff --git a/ts/routes/deck-options/FsrsOptions.svelte b/ts/routes/deck-options/FsrsOptions.svelte
index 569dee9ddc9..319095e3e7e 100644
--- a/ts/routes/deck-options/FsrsOptions.svelte
+++ b/ts/routes/deck-options/FsrsOptions.svelte
@@ -25,7 +25,6 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
     import SettingTitle from "$lib/components/SettingTitle.svelte";
     import SwitchRow from "$lib/components/SwitchRow.svelte";
 
-    import DateInput from "./DateInput.svelte";
     import GlobalLabel from "./GlobalLabel.svelte";
     import type { DeckOptionsState } from "./lib";
     import SpinBoxFloatRow from "./SpinBoxFloatRow.svelte";
@@ -374,11 +373,6 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
         placeholder={defaultWeightSearch}
     />
 
-    <DateInput bind:date={$config.ignoreRevlogsBeforeDate}>
-        <SettingTitle on:click={() => openHelpModal("ignoreBefore")}>
-            {tr.deckConfigIgnoreBefore()}
-        </SettingTitle>
-    </DateInput>
     <button
         class="btn {computingWeights ? 'btn-warning' : 'btn-primary'}"
         disabled={!computingWeights && computing}
diff --git a/ts/routes/deck-options/FsrsOptionsOuter.svelte b/ts/routes/deck-options/FsrsOptionsOuter.svelte
index fe9c4cb046c..06a50b3cb9d 100644
--- a/ts/routes/deck-options/FsrsOptionsOuter.svelte
+++ b/ts/routes/deck-options/FsrsOptionsOuter.svelte
@@ -45,11 +45,6 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
                 tr.deckConfigComputeOptimalWeightsTooltip2(),
             sched: HelpItemScheduler.FSRS,
         },
-        ignoreRevlogsBeforeMs: {
-            title: tr.deckConfigIgnoreBefore(),
-            help: tr.deckConfigIgnoreBeforeTooltip2(),
-            sched: HelpItemScheduler.FSRS,
-        },
         rescheduleCardsOnChange: {
             title: tr.deckConfigRescheduleCardsOnChange(),
             help: tr.deckConfigRescheduleCardsOnChangeTooltip(),