open-git/nofx
1
0
Fork 0
mirror of https://github.com/NoFxAiOS/nofx.git synced 2025-12-06 13:54:41 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: ensure tab navigation updates page state correctly

Browse Source 
Tab buttons were only calling navigate() which changes URL but doesn't
trigger popstate events. App.tsx listens to popstate/hashchange to
update page state, so clicks appeared to do nothing.

Now all tab buttons call both onPageChange() callback and navigate()
to ensure page state updates and URL stays in sync.
This commit is contained in:
tinkle-community
2025-11-30 12:40:14 +08:00
parent b71272d48b
commit 2e964d704a
1 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
web/src/components/HeaderBar.tsx
View File

@@ -101,6 +101,9 @@ export default function HeaderBar({
<>
<button
onClick={() => {
if (onPageChange) {
onPageChange('competition')
}
navigate('/competition')
}}
className="text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500"
@@ -140,6 +143,9 @@ export default function HeaderBar({
<button
onClick={() => {
if (onPageChange) {
onPageChange('traders')
}
navigate('/traders')
}}
className="text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500"
@@ -179,6 +185,9 @@ export default function HeaderBar({
<button
onClick={() => {
if (onPageChange) {
onPageChange('trader')
}
navigate('/dashboard')
}}
className="text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500"
@@ -261,9 +270,8 @@ export default function HeaderBar({
onClick={() => {
if (onPageChange) {
onPageChange('faq')
} else {
navigate('/faq')
}
navigate('/faq')
}}
className="text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500"
style={{
@@ -708,9 +716,8 @@ export default function HeaderBar({
onClick={() => {
if (onPageChange) {
onPageChange('traders')
} else {
navigate('/traders')
}
navigate('/traders')
setMobileMenuOpen(false)
}}
className="block text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500 hover:text-yellow-500"
@@ -743,9 +750,8 @@ export default function HeaderBar({
onClick={() => {
if (onPageChange) {
onPageChange('trader')
} else {
navigate('/dashboard')
}
navigate('/dashboard')
setMobileMenuOpen(false)
}}
className="block text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500 hover:text-yellow-500"
@@ -778,9 +784,8 @@ export default function HeaderBar({
onClick={() => {
if (onPageChange) {
onPageChange('faq')
} else {
navigate('/faq')
}
navigate('/faq')
setMobileMenuOpen(false)
}}
className="block text-sm font-bold transition-all duration-300 relative focus:outline-2 focus:outline-yellow-500 hover:text-yellow-500"