Skip to content

Commit 5d48638

Browse files
authored
fix: comment top sort icon should be trophy (#2094)
1 parent e89090b commit 5d48638

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

src/features/feed/sort/topSorts.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import { barChartOutline } from "ionicons/icons";
2+
13
import {
24
ALL_DURATIONS,
35
createSortFromDurations,
@@ -8,13 +10,18 @@ export const TOP_DURATIONS = ALL_DURATIONS satisfies readonly VgerDuration[];
810

911
const TOP_DURATION_SORTS = createSortFromDurations("Top", TOP_DURATIONS);
1012

11-
export const LEMMY_TOP_SORTS = {
13+
const TOP = {
1214
label: "Top",
15+
icon: barChartOutline,
16+
} as const;
17+
18+
export const LEMMY_TOP_SORTS = {
19+
...TOP,
1320
children: TOP_DURATION_SORTS,
1421
} as const;
1522

1623
export const PIEFED_TOP_SORTS = {
17-
label: "Top",
24+
...TOP,
1825
children: [
1926
"TopHour",
2027
"TopSixHour",

src/routes/pages/shared/Sort.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import { startCase } from "es-toolkit";
55
import {
66
albumsOutline,
77
arrowUpCircleOutline,
8-
barChartOutline,
98
calendarOutline,
109
chatbubbleEllipsesOutline,
1110
chatbubblesOutline,
@@ -291,9 +290,8 @@ export function getSortIcon(sort: AnyVgerSort): string {
291290
case "TopYear":
292291
return calendarYear;
293292
case "TopAll":
294-
return trophyOutline;
295293
case "Top":
296-
return barChartOutline;
294+
return trophyOutline;
297295
case "Old":
298296
return hourglassOutline;
299297
case "Controversial":

0 commit comments

Comments
 (0)