Skip to content

Commit 7fa3c00

Browse files
committed
refactor: 상세 보기의 경우 url 추가 제거
1 parent f8bc39f commit 7fa3c00

2 files changed

Lines changed: 5 additions & 34 deletions

File tree

src/app/ranking/page.tsx

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
"use client"
22

3-
import { useState, useEffect, Suspense, useRef } from "react"
3+
import { useState, Suspense, useRef } from "react"
44
import { useRouter, useSearchParams } from "next/navigation"
55
import { LazyMotion, domAnimation, m } from "framer-motion"
66
import { useRankingList } from "@/features/ranking/api/ranking-service"
@@ -84,13 +84,7 @@ function RankingContent() {
8484
const tierParam = searchParams.get('tier')
8585
const selectedTier: Tier | 'ALL' = tierParam && TIERS.includes(tierParam as Tier) ? tierParam as Tier : 'ALL'
8686

87-
useEffect(() => {
88-
const userParam = searchParams.get('user')
89-
if (userParam) {
90-
setSelectedUsername(userParam)
91-
setModalOpen(true)
92-
}
93-
}, [searchParams])
87+
9488

9589
const { data, isLoading } = useRankingList(
9690
page,
@@ -147,19 +141,12 @@ function RankingContent() {
147141
if (!username) return;
148142
setSelectedUsername(username)
149143
setModalOpen(true)
150-
const params = new URLSearchParams(searchParams.toString())
151-
params.set('user', username)
152-
router.push(`?${params.toString()}`, { scroll: false })
153144
}
154145

155146
const handleModalClose = (open: boolean) => {
156147
setModalOpen(open)
157148
if (!open) {
158149
setSelectedUsername(null)
159-
const params = new URLSearchParams(searchParams.toString())
160-
params.delete('user')
161-
const newUrl = params.toString() ? `?${params.toString()}` : window.location.pathname
162-
router.push(newUrl, { scroll: false })
163150
}
164151
}
165152

src/features/ranking/components/ranking-section.tsx

Lines changed: 3 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
"use client"
22

3-
import { useState, useEffect, useRef, memo, useCallback } from "react"
4-
import { useRouter, useSearchParams } from "next/navigation"
3+
import { useState, useRef, memo, useCallback } from "react"
54
import { motion } from "framer-motion"
65
import { useRankingList } from "../api/ranking-service"
76
import { usePrefetchUser } from "@/features/user/api/user-service"
@@ -141,8 +140,6 @@ const DesktopRankingRow = memo(function DesktopRankingRow({ user, onUserClick, o
141140
})
142141

143142
export function RankingSection() {
144-
const router = useRouter()
145-
const searchParams = useSearchParams()
146143
const [page, setPage] = useState(0)
147144
const [pageInput, setPageInput] = useState("")
148145
const [selectedTier, setSelectedTier] = useState<Tier | 'ALL'>('ALL')
@@ -162,13 +159,7 @@ export function RankingSection() {
162159
const pageInfo = data?.pageInfo;
163160
const totalPages = pageInfo?.totalPages || 1;
164161

165-
useEffect(() => {
166-
const userParam = searchParams.get('user')
167-
if (userParam) {
168-
setSelectedUsername(userParam)
169-
setModalOpen(true)
170-
}
171-
}, [searchParams])
162+
172163

173164
// ✅ Scroll to list top when page changes
174165
const scrollToList = () => {
@@ -211,19 +202,12 @@ export function RankingSection() {
211202
const handleUserClick = useCallback((username: string) => {
212203
setSelectedUsername(username)
213204
setModalOpen(true)
214-
const params = new URLSearchParams(searchParams.toString())
215-
params.set('user', username)
216-
router.push(`?${params.toString()}`, { scroll: false })
217-
}, [router, searchParams])
205+
}, [])
218206

219207
const handleModalClose = (open: boolean) => {
220208
setModalOpen(open)
221209
if (!open) {
222210
setSelectedUsername(null)
223-
const params = new URLSearchParams(searchParams.toString())
224-
params.delete('user')
225-
const newUrl = params.toString() ? `?${params.toString()}` : window.location.pathname
226-
router.push(newUrl, { scroll: false })
227211
}
228212
}
229213

0 commit comments

Comments
 (0)