From 928b69f415d1f4331a89969d888140dba8109985 Mon Sep 17 00:00:00 2001 From: midzelis Date: Mon, 3 Nov 2025 22:09:05 +0000 Subject: [PATCH] refactor(web): rename DayGroup/MonthGroup to TimelineDay/TimelineMonth MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Rename classes: DayGroup → TimelineDay, MonthGroup → TimelineMonth - Use short variable names: dayGroup → day, monthGroup → month - Update all method names and properties for consistency - Convert relative imports to $lib alias convention No functional changes. --- .../components/album-page/album-viewer.svelte | 2 +- .../lib/components/timeline/Scrubber.svelte | 16 +- .../lib/components/timeline/Timeline.svelte | 112 ++++++------ .../timeline/TimelineAssetViewer.svelte | 2 +- .../timeline/TimelineDateGroup.svelte | 103 +++++------ .../timeline/actions/SelectAllAction.svelte | 2 +- .../actions/TimelineKeyboardActions.svelte | 2 +- .../timeline/actions/focus-actions.ts | 2 +- .../ScrollSegment.svelte.ts | 1 + ...-group.svelte.ts => TimelineDay.svelte.ts} | 34 ++-- ...spec.ts => TimelineManager.svelte.spec.ts} | 132 +++++++-------- ...er.svelte.ts => TimelineManager.svelte.ts} | 160 +++++++++--------- ...roup.svelte.ts => TimelineMonth.svelte.ts} | 127 +++++++------- .../group-insertion-cache.svelte.ts | 56 +++--- .../internal/TestHooks.svelte.ts | 12 +- .../internal/intersection-support.svelte.ts | 20 +-- .../internal/layout-support.svelte.ts | 50 +++--- .../internal/load-support.svelte.ts | 16 +- .../internal/search-support.svelte.spec.ts | 26 +-- .../internal/search-support.svelte.ts | 76 ++++----- .../internal/websocket-support.svelte.ts | 2 +- .../managers/timeline-manager/utils.svelte.ts | 2 +- .../timeline-manager/viewer-asset.svelte.ts | 17 +- web/src/lib/modals/NavigateToDateModal.svelte | 2 +- web/src/lib/utils/actions.ts | 2 +- web/src/lib/utils/asset-utils.ts | 12 +- web/src/lib/utils/timeline-util.spec.ts | 34 ++-- web/src/lib/utils/timeline-util.ts | 4 +- .../[[assetId=id]]/+page.svelte | 4 +- .../[[assetId=id]]/+page.svelte | 2 +- .../[[assetId=id]]/+page.svelte | 2 +- .../[[assetId=id]]/+page.svelte | 2 +- .../[[assetId=id]]/+page.svelte | 2 +- .../(user)/photos/[[assetId=id]]/+page.svelte | 2 +- .../[[assetId=id]]/+page.svelte | 2 +- .../[[assetId=id]]/+page.svelte | 2 +- .../(user)/utilities/geolocation/+page.svelte | 10 +- 37 files changed, 517 insertions(+), 537 deletions(-) create mode 100644 web/src/lib/managers/VirtualScrollManager/ScrollSegment.svelte.ts rename web/src/lib/managers/timeline-manager/{day-group.svelte.ts => TimelineDay.svelte.ts} (84%) rename web/src/lib/managers/timeline-manager/{timeline-manager.svelte.spec.ts => TimelineManager.svelte.spec.ts} (84%) rename web/src/lib/managers/timeline-manager/{timeline-manager.svelte.ts => TimelineManager.svelte.ts} (78%) rename web/src/lib/managers/timeline-manager/{month-group.svelte.ts => TimelineMonth.svelte.ts} (74%) diff --git a/web/src/lib/components/album-page/album-viewer.svelte b/web/src/lib/components/album-page/album-viewer.svelte index 8d169fbc8f..269bad1d23 100644 --- a/web/src/lib/components/album-page/album-viewer.svelte +++ b/web/src/lib/components/album-page/album-viewer.svelte @@ -6,7 +6,7 @@ import SelectAllAssets from '$lib/components/timeline/actions/SelectAllAction.svelte'; import AssetSelectControlBar from '$lib/components/timeline/AssetSelectControlBar.svelte'; import Timeline from '$lib/components/timeline/Timeline.svelte'; - import { TimelineManager } from '$lib/managers/timeline-manager/timeline-manager.svelte'; + import { TimelineManager } from '$lib/managers/timeline-manager/TimelineManager.svelte'; import { AssetInteraction } from '$lib/stores/asset-interaction.svelte'; import { assetViewingStore } from '$lib/stores/asset-viewing.store'; import { dragAndDropFilesStore } from '$lib/stores/drag-and-drop-files.store'; diff --git a/web/src/lib/components/timeline/Scrubber.svelte b/web/src/lib/components/timeline/Scrubber.svelte index d64d45cc87..756aa4bbc1 100644 --- a/web/src/lib/components/timeline/Scrubber.svelte +++ b/web/src/lib/components/timeline/Scrubber.svelte @@ -1,5 +1,5 @@ @@ -622,23 +622,23 @@ {/if} - {#each timelineManager.months as monthGroup (monthGroup.viewId)} - {@const display = monthGroup.intersecting} - {@const absoluteHeight = monthGroup.top} + {#each timelineManager.months as month (month.viewId)} + {@const display = month.intersecting} + {@const absoluteHeight = month.top} - {#if !monthGroup.isLoaded} + {#if !month.isLoaded}
- +
{:else if display}
handleGroupSelect(timelineManager, title, assets)} onSelectAssetCandidates={handleSelectAssetCandidates} onSelectAssets={handleSelectAssets} diff --git a/web/src/lib/components/timeline/TimelineAssetViewer.svelte b/web/src/lib/components/timeline/TimelineAssetViewer.svelte index a121bd1938..43ab6a094f 100644 --- a/web/src/lib/components/timeline/TimelineAssetViewer.svelte +++ b/web/src/lib/components/timeline/TimelineAssetViewer.svelte @@ -2,7 +2,7 @@ import type { Action } from '$lib/components/asset-viewer/actions/action'; import { AssetAction } from '$lib/constants'; import { authManager } from '$lib/managers/auth-manager.svelte'; - import { TimelineManager } from '$lib/managers/timeline-manager/timeline-manager.svelte'; + import { TimelineManager } from '$lib/managers/timeline-manager/TimelineManager.svelte'; import { assetViewingStore } from '$lib/stores/asset-viewing.store'; import { updateStackedAssetInTimeline, updateUnstackedAssetInTimeline } from '$lib/utils/actions'; import { navigate } from '$lib/utils/navigation'; diff --git a/web/src/lib/components/timeline/TimelineDateGroup.svelte b/web/src/lib/components/timeline/TimelineDateGroup.svelte index c662c16e72..a19ef4a118 100644 --- a/web/src/lib/components/timeline/TimelineDateGroup.svelte +++ b/web/src/lib/components/timeline/TimelineDateGroup.svelte @@ -1,19 +1,17 @@ -{#each filterIntersecting(monthGroup.dayGroups) as dayGroup, groupIndex (dayGroup.day)} - {@const absoluteWidth = dayGroup.left} +{#each filterIntersecting(month.days) as day, groupIndex (day.day)} + {@const absoluteWidth = day.left}
{ isMouseOverGroup = true; - assetMouseEventHandler(dayGroup.groupTitle, null); + assetMouseEventHandler(day.dayTitle, null); }} onmouseleave={() => { isMouseOverGroup = false; - assetMouseEventHandler(dayGroup.groupTitle, null); + assetMouseEventHandler(day.dayTitle, null); }} >
{#if !singleSelect}
handleSelectGroup(dayGroup.groupTitle, assetsSnapshot(dayGroup.getAssets()))} - onkeydown={() => handleSelectGroup(dayGroup.groupTitle, assetsSnapshot(dayGroup.getAssets()))} + class:w-8={(hoveredDay === day.dayTitle && isMouseOverGroup) || + assetInteraction.selectedGroup.has(day.dayTitle)} + onclick={() => handleSelectGroup(day.dayTitle, assetsSnapshot(day.getAssets()))} + onkeydown={() => handleSelectGroup(day.dayTitle, assetsSnapshot(day.getAssets()))} > - {#if assetInteraction.selectedGroup.has(dayGroup.groupTitle)} + {#if assetInteraction.selectedGroup.has(day.dayTitle)} {:else} @@ -177,19 +171,14 @@
{/if} - - {dayGroup.groupTitle} + + {day.dayTitle}
-
- {#each filterIntersecting(dayGroup.viewerAssets) as viewerAsset (viewerAsset.id)} +
+ {#each filterIntersecting(day.viewerAssets) as viewerAsset (viewerAsset.id)} {@const position = viewerAsset.position!} {@const asset = viewerAsset.asset!} @@ -212,17 +201,17 @@ {groupIndex} onClick={(asset) => { if (typeof onThumbnailClick === 'function') { - onThumbnailClick(asset, timelineManager, dayGroup, _onClick); + onThumbnailClick(asset, timelineManager, day, _onClick); } else { - _onClick(timelineManager, dayGroup.getAssets(), dayGroup.groupTitle, asset); + _onClick(timelineManager, day.getAssets(), day.dayTitle, asset); } }} - onSelect={(asset) => assetSelectHandler(timelineManager, asset, dayGroup.getAssets(), dayGroup.groupTitle)} - onMouseEvent={() => assetMouseEventHandler(dayGroup.groupTitle, assetSnapshot(asset))} + onSelect={(asset) => assetSelectHandler(timelineManager, asset, day.getAssets(), day.dayTitle)} + onMouseEvent={() => assetMouseEventHandler(day.dayTitle, assetSnapshot(asset))} selected={assetInteraction.hasSelectedAsset(asset.id) || - dayGroup.monthGroup.timelineManager.albumAssets.has(asset.id)} + day.month.timelineManager.albumAssets.has(asset.id)} selectionCandidate={assetInteraction.hasSelectionCandidate(asset.id)} - disabled={dayGroup.monthGroup.timelineManager.albumAssets.has(asset.id)} + disabled={day.month.timelineManager.albumAssets.has(asset.id)} thumbnailWidth={position.width} thumbnailHeight={position.height} /> diff --git a/web/src/lib/components/timeline/actions/SelectAllAction.svelte b/web/src/lib/components/timeline/actions/SelectAllAction.svelte index 883ec1d54e..7cd64fccda 100644 --- a/web/src/lib/components/timeline/actions/SelectAllAction.svelte +++ b/web/src/lib/components/timeline/actions/SelectAllAction.svelte @@ -1,5 +1,5 @@