Skip to content

Commit b2f0d30

Browse files
Tony QiuTony Qiu
authored andcommitted
fix eventstatus text
1 parent c4b95ae commit b2f0d30

File tree

3 files changed

+16
-21
lines changed

3 files changed

+16
-21
lines changed

frontend/src/features/events/components/EventsStatusBar.tsx

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,34 +4,27 @@ interface EventsStatusBarProps {
44
isLoading: boolean;
55
searchTerm: string;
66
isShowingPastEvents: boolean;
7-
paginatedCount: number;
87
totalCount: number;
9-
view: "grid" | "calendar";
108
}
119

1210
const EventsStatusBar: React.FC<EventsStatusBarProps> = ({
1311
isLoading,
1412
searchTerm,
1513
isShowingPastEvents,
16-
paginatedCount,
1714
totalCount,
18-
view,
1915
}) => {
2016
const getStatusText = () => {
21-
if (isLoading) return "Loading...";
22-
23-
// In calendar view, always show total count since pagination doesn't apply
24-
const displayCount = view === "calendar" ? totalCount : paginatedCount;
17+
if (isLoading) return "Loading...";
2518

2619
if (searchTerm) {
27-
return `Showing ${displayCount} of ${totalCount} found events`;
20+
return `${totalCount} found events`;
2821
}
2922

3023
if (isShowingPastEvents) {
31-
return `Showing ${displayCount} of ${totalCount} total events`;
24+
return `${totalCount} total events`;
3225
}
3326

34-
return `Showing ${displayCount} of ${totalCount} upcoming events`;
27+
return `${totalCount} upcoming events`;
3528
};
3629

3730
return <p className="text-sm">{getStatusText()}</p>;

frontend/src/features/events/hooks/useEvents.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,16 +127,18 @@ export function useEvents() {
127127

128128
if (searchTerm) {
129129
title = `${events.length} Found Events - Wat2Do`;
130+
} else if (startDate) {
131+
title = `${events.length} Total Events - Wat2Do`;
130132
} else {
131-
title = `${events.length} Events - Wat2Do`;
133+
title = `${events.length} Upcoming Events - Wat2Do`;
132134
}
133135

134136
if (!isLoadingData) {
135137
previousTitleRef.current = title;
136138
}
137139

138140
return previousTitleRef.current;
139-
}, [events.length, isLoading, searchTerm, hasActiveFilters]);
141+
}, [events.length, isLoading, searchTerm, hasActiveFilters, startDate]);
140142

141143
useDocumentTitle(documentTitle);
142144

frontend/src/features/events/pages/EventsPage.tsx

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,14 @@ function EventsPage() {
1717
const [searchParams, setSearchParams] = useSearchParams();
1818
const view = (searchParams.get("view") as "grid" | "calendar") || "grid";
1919

20-
const handleViewChange = useCallback((newView: "grid" | "calendar") => {
21-
const newParams = new URLSearchParams(searchParams);
22-
newParams.set("view", newView);
23-
setSearchParams(newParams);
24-
}, [searchParams, setSearchParams]);
20+
const handleViewChange = useCallback(
21+
(newView: "grid" | "calendar") => {
22+
const newParams = new URLSearchParams(searchParams);
23+
newParams.set("view", newView);
24+
setSearchParams(newParams);
25+
},
26+
[searchParams, setSearchParams]
27+
);
2528

2629
const {
2730
data,
@@ -32,7 +35,6 @@ function EventsPage() {
3235
handleToggleStartDate,
3336
} = useEvents();
3437

35-
3638
const {
3739
isSelectMode,
3840
selectedEvents,
@@ -68,9 +70,7 @@ function EventsPage() {
6870
isLoading={isLoading}
6971
searchTerm={searchTerm}
7072
isShowingPastEvents={isShowingPastEvents}
71-
paginatedCount={data.length}
7273
totalCount={data.length}
73-
view={view}
7474
/>
7575
</div>
7676

0 commit comments

Comments
 (0)