訪問日の「過去1年」を期間だけのトグルにし、これだけを表示と独立させる

見出し行の2つのトグルを別々の軸にした。「過去1年」が決めるのは対象の期間
(1年前〜今日)、「これだけを表示」が決めるのは他のスポットを隠すかどうかで、
同時に点けられる。排他にしないのは、片方だけで使う場面が両方あるため ――
期間だけ広げて周りのスポットも見たいことも、今日の経路だけに絞りたいこともある。

「過去1年」は handleSelectVisitDate を呼ぶだけで、isolate には触れない。
点いているかは専用のフラグではなく期間そのもの(対象日がちょうど1年前〜今日か)を
見て決める。フラグを別に持つと、カレンダー側の選択と食い違ったときに
どちらが正しいのか決められなくなる。

解除したときに既定(今日)へ戻すのは変えていない。期間だけが1年のまま残ると、
1年ぶんの訪問が1本の訪問順の経路として繋がり、地図が矢印だらけになって読めなくなる。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: claude

CLAUDE.md

@@ -366,11 +366,13 @@ PATCH `/api/spot-types/[id]`は`series_styles`・`categories`を保存前に検
 
 選び方は**カレンダー**(`components/VisitDateCalendar.tsx`)。絞り込みモーダルの訪問日セクションには**選択中の日(期間)を出すボタン**だけを置き、押すと**カレンダーを別モーダル(z-[60]。絞り込みモーダルのz-50の上)で開く** —— 絞り込みモーダルにカレンダーを直に置くと、他の条件を見るのに毎回その分スクロールすることになるため。**カレンダーは日を選んでも閉じない**(期間は2回のタップで決まるので、1回目で閉じると期間を選べない)。選択はその場で反映されるので、閉じる操作は「閉じる」だけでよい。
 
-**よく使う期間はカレンダーを開かずに選べる。** 訪問日セクションとカレンダーに**「今日」「表示しない」**を並べ、見出し行の「これだけを表示」の隣に**「過去1年だけを表示」**を置いてある(`MapView`の`visitDateOptions`)。**1年前の日付は`Date`に年だけ引かせる**ので、2月29日は3月1日に送られる(閏年でない年に2月29日は無い)——絞り込みの範囲としては1日の差なので、月末を特別扱いはしない。
+**よく使う期間はカレンダーを開かずに選べる。** 訪問日セクションとカレンダーに**「今日」「表示しない」**を並べ、見出し行に**「過去1年」**のトグルを置いてある(`MapView`の`visitDateOptions`)。**1年前の日付は`Date`に年だけ引かせる**ので、2月29日は3月1日に送られる(閏年でない年に2月29日は無い)——絞り込みの範囲としては1日の差なので、月末を特別扱いはしない。
 
-**「これだけを表示」と「過去1年だけを表示」は、同じ絞り(`isolate: "visit"`)の入り口が2つあるだけ**で、違うのは対象の期間だけ。押すと対象日を**1年前〜今日**にしてから絞りを掛けるので、期間を選ぶ操作が要らない(カレンダーだと開始月まで12回さかのぼって2回タップすることになる)。**`isolate`に値を増やしていない** —— 増やすと絞り込みを効かせるかの判定(`skipFiltersInIsolate`)や保存値の読み込みまで、期間の違いでしかないものを各所で分岐させることになる。期間は`visitedDate`/`visitedDateTo`が既に持っているので、**押されている側は対象日がちょうど1年前〜今日か(`isPastYearRange`)で見分ける**。この作りのおかげで2つは自動的に排他になる(条件が互いの否定になっている)。カレンダーで偶然その期間を選んだときも「過去1年だけを表示」が点くが、実際にその期間なので嘘にはならない。
+**見出し行の2つのトグルは別々の軸。** 「過去1年」が決めるのは**対象の期間**(1年前〜今日)、「これだけを表示」が決めるのは**他のスポットを隠すかどうか**(`isolate`)で、**同時に点けられる**(過去1年に訪問したスポットだけを出す)。排他にしていないのは、片方だけで使う場面が両方あるため——期間だけ広げて周りのスポットも見たいことも、今日の経路だけに絞りたいこともある。
 
-**「過去1年だけを表示」を解除すると、対象日も既定(今日)へ戻す。** 絞りだけ解いて期間を1年のまま残すと、**1年ぶんの訪問が1本の訪問順の経路として繋がり**、地図が矢印だらけになって読めなくなる。
+**「過去1年」が点いているかは、専用のフラグではなく期間そのものを見て決める**(`isPastYearRange`=対象日がちょうど1年前〜今日か)。カレンダーで同じ期間を選んだときも点くが、実際にその期間なので嘘にはならない。フラグを別に持つと、カレンダー側の選択と食い違ったときにどちらが正しいのか決められなくなる。
+
+**「過去1年」を解除すると既定(今日)へ戻す。** 期間だけが1年のまま残ると、**1年ぶんの訪問が1本の訪問順の経路として繋がり**、地図が矢印だらけになって読めなくなる。
 
 **訪問記録のある日には日付の下に緑の点**を打つ(`visitDateSet`。どの日に記録があるか分からないまま総当たりで選ぶことになるのを避けるため。**他のスポット種別への訪問も数える** — 経路が種別をまたぐようになったので、その日を落とすと辿れない)。1回目のタップで開始日、2回目で終了日。既に期間が決まっている状態でのタップは新しい開始日として選び直し(範囲を狭めるのにリセットを挟まずに済む)、開始日より前をタップしたときはその日を開始日にして元の開始日を終了日にする(前方向にも伸ばせる)。同じ日を2回タップしても単日のまま。「今日」「表示しない」はよく使うので、カレンダーの中と絞り込みモーダルの両方に置く。既定値は今日。かつてはセレクト(今日 / 表示しない / 訪問のある日の一覧)だったが、期間指定とデータのある日の可視化のためカレンダーに変えた。「表示しない」(`null`)は`saveFilters`が文字列`"none"`で保存し、`loadSavedFilters`は`"none"`のときだけ`null`=表示しないにする。**「今日」(値が`todayKey()`と一致)は具体的な日付ではなく文字列`"today"`で保存し、読み込み時にその日の今日へ解決する**(日付のまま保存すると翌日に開いたとき前日が選ばれた状態で復元されてしまうため)。日付はその日、それ以外(旧仕様の絞り込みだった頃の`null`・キー欠落・不正値)は今日に倒すため、**既存ユーザーも初回から今日の経路が出る**(`todayKey`/`defaultMapFilters`)。終了日(`visitedDateTo`)は**「今日」のような相対表現を持たず具体的な日付でだけ保存する**(終了日だけ動くと期間の長さが日をまたぐたびに変わってしまうため)。カレンダーで日を選んだとき(`handleSelectVisitDate`)は、対象日(期間)をセットしたうえで**その経路全体が画面に収まるよう`fitBounds`する**(1地点だけならmaxZoomまで寄る。経路が0件・「表示しない」のときは地図を動かさない)。開始日を`null`にするときは終了日も一緒に落とす(残っていると次に日を選んだとき意図しない期間になる)。ユーザーが明示的に選んだときだけ移動し、マウント時の既定(今日)の復元では移動しない。日付キーへの変換は`toVisitDateKey`(`visits.visited_on`はtimestamptzでUTC文字列のため、**必ずローカル時刻で日を切る** — UTCのまま切ると日本時間の朝9時前の訪問が前日になる)。
 

README.md

@@ -20,7 +20,7 @@
 
 | 機能 | 説明 |
 |---|---|
-| **絞り込み** | ランク・シリーズ・訪問状態・カテゴリ・訪問日で絞れる。訪問日は「今日」をカレンダーを開かずに選べ、**「過去1年だけを表示」**でこの1年に訪問したスポットだけに絞れる |
+| **絞り込み** | ランク・シリーズ・訪問状態・カテゴリ・訪問日で絞れる。訪問日は「今日」「過去1年」をカレンダーを開かずに選べる |
 | **見た目の3軸** | **ランク**(A〜E。色と大きさ)・**シリーズ**(1スポットに1つ。ピンの中のアイコンと形)・**カテゴリ**(複数可。絞り込み専用) |
 | **訪問予定** | 行きたい場所のブックマーク。訪問を記録すると自動的に外れる |
 | **訪問予定リスト** | 複数スポットを順序付きでまとめた旅程。地図に紫の矢印で経路を描き、Googleマップの経路検索へそのまま渡せる。回り終わったリストは**アーカイブ**して一覧から下げられる(中身は残り、スポット画面の「アーカイブ」から読み直せる) |

components/MapView.tsx

@@ -1564,15 +1564,13 @@ export default function MapView({
     () => ({ today: todayKey(), oneYearAgo: oneYearAgoKey() }),
     []
   );
-  // 訪問日の「これだけを表示」中か。「これだけを表示」と「過去1年だけを表示」は
-  // 同じ絞り(isolate: "visit")の入り口が2つあるだけで、違うのは対象の期間。
-  // **isolateに値を増やしていない** —— 増やすと絞り込みを効かせるかの判定
-  // (skipFiltersInIsolate)や保存値の読み込みまで、期間の違いでしかないものを
-  // 各所で分岐させることになる。期間は visitedDate/visitedDateTo が既に持っている
+  // 訪問日セクションの2つのトグルの状態。**別々の軸**で、片方が対象の期間を、
+  // もう片方が他のスポットを隠すかどうかを決める(同時に点けられる)。
   const isolatingVisit = filters.isolate === "visit";
-  // 対象日がちょうど「1年前〜今日」か。押されているボタンをこれで見分ける
-  // (カレンダーで同じ期間を選んだときも「過去1年だけを表示」が点くが、
-  //  実際にその期間なので嘘にはならない)
+  // 対象日がちょうど「1年前〜今日」か=「過去1年」が点いているか。
+  // **期間そのものを見て決める**ので、カレンダーで同じ期間を選んだときも点く
+  // (実際にその期間なので嘘にはならない)。専用のフラグを持つと、
+  // カレンダー側の選択と食い違ったときにどちらが正しいのか決められなくなる
   const isPastYearRange =
     filters.visitedDate === visitDateOptions.oneYearAgo &&
     filters.visitedDateTo === visitDateOptions.today;
@@ -3835,60 +3833,50 @@ export default function MapView({
                   </HelpTip>
                 </p>
                 <div className="flex shrink-0 items-center gap-1.5">
-                  {/* 「これだけを表示」と「過去1年だけを表示」は同じ絞り(isolate)の
-                      入り口が2つあるだけで、違うのは対象の期間。どちらか一方しか
-                      点かないので、押されている側を見れば期間が読める */}
-                  {/* その日のスポットだけに絞る(他のスポット・ルート・訪問予定リストは隠す) */}
+                  {/* 対象日を1年前〜今日にするトグル。押すだけで期間の選択が要らない
+                      (カレンダーだと開始月まで12回さかのぼって2回タップになる)。
+                      **「これだけを表示」とは別の軸**で、こちらが決めるのは対象の期間、
+                      あちらが決めるのは他のスポットを隠すかどうか。
+                      両方を同時に点けられる(過去1年に訪問したスポットだけを出す)。
+                      解除すると既定=今日へ戻す —— 期間だけが1年のまま残ると、
+                      1年ぶんの訪問が1本の経路として繋がって読めなくなる */}
                   <button
                     type="button"
-                    disabled={!filters.visitedDate}
-                    aria-pressed={isolatingVisit && !isPastYearRange}
+                    aria-pressed={isPastYearRange}
                     onClick={() =>
-                      setFilters({
-                        ...filters,
-                        isolate: isolatingVisit ? null : "visit",
-                      })
+                      handleSelectVisitDate(
+                        isPastYearRange
+                          ? visitDateOptions.today
+                          : visitDateOptions.oneYearAgo,
+                        isPastYearRange ? null : visitDateOptions.today
+                      )
                     }
-                    className={`shrink-0 rounded-full border px-2.5 py-0.5 text-xs font-medium disabled:opacity-40 ${
-                      isolatingVisit && !isPastYearRange
+                    className={`shrink-0 rounded-full border px-2.5 py-0.5 text-xs font-medium ${
+                      isPastYearRange
                         ? "border-blue-600 bg-blue-600 text-white"
                         : "border-gray-300 bg-white text-gray-500"
                     }`}
                   >
-                    これだけを表示
+                    過去1年
                   </button>
-                  {/* 過去1年に訪問したスポットだけに絞る。対象日を1年前〜今日にしてから
-                      同じ絞りを掛けるので、押すだけで期間の選択が要らない
-                      (カレンダーだと開始月まで12回さかのぼることになる)。
-                      解除すると対象日も既定=今日へ戻す —— 期間だけが1年のまま残ると、
-                      1年ぶんの訪問が1本の経路として繋がって読めなくなる */}
+                  {/* その日のスポットだけに絞る(他のスポット・ルート・訪問予定リストは隠す) */}
                   <button
                     type="button"
-                    aria-pressed={isolatingVisit && isPastYearRange}
+                    disabled={!filters.visitedDate}
+                    aria-pressed={isolatingVisit}
                     onClick={() =>
-                      setFilters(
-                        isolatingVisit && isPastYearRange
-                          ? {
-                              ...filters,
-                              visitedDate: visitDateOptions.today,
-                              visitedDateTo: null,
-                              isolate: null,
-                            }
-                          : {
-                              ...filters,
-                              visitedDate: visitDateOptions.oneYearAgo,
-                              visitedDateTo: visitDateOptions.today,
-                              isolate: "visit",
-                            }
-                      )
+                      setFilters({
+                        ...filters,
+                        isolate: isolatingVisit ? null : "visit",
+                      })
                     }
-                    className={`shrink-0 rounded-full border px-2.5 py-0.5 text-xs font-medium ${
-                      isolatingVisit && isPastYearRange
+                    className={`shrink-0 rounded-full border px-2.5 py-0.5 text-xs font-medium disabled:opacity-40 ${
+                      isolatingVisit
                         ? "border-blue-600 bg-blue-600 text-white"
                         : "border-gray-300 bg-white text-gray-500"
                     }`}
                   >
-                    過去1年だけを表示
+                    これだけを表示
                   </button>
                   {/* このセクションだけのリセット(対象日を既定=今日に戻し、
                       「これだけを表示」も解除する) */}