Quitar URLs javascript: (React 19 las bloquea)
React 19 bloquea href="javascript:void(0)". SiteHeader: DESCONECTAR ahora enlaza
a /{locale}/log-out (ruta real), el menú hamburguesa y el logo pequeño usan
role/button o href real. PostActions (foro): editar/eliminar pasan a role=button.
Se elimina el logout por fetch del header (lo hace la ruta /log-out).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -120,11 +120,11 @@ export function PostActions({
|
|||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="separate small-font">
|
<div className="separate small-font">
|
||||||
<a href="javascript:void(0);" onClick={() => setEditing(true)} className="second-brown">
|
<a role="button" tabIndex={0} onClick={() => setEditing(true)} className="second-brown" style={{ cursor: 'pointer' }}>
|
||||||
{t('edit')}
|
{t('edit')}
|
||||||
</a>
|
</a>
|
||||||
{' '}
|
{' '}
|
||||||
<a href="javascript:void(0);" onClick={() => { if (!busy) remove() }} className="second-brown">
|
<a role="button" tabIndex={0} onClick={() => { if (!busy) remove() }} className="second-brown" style={{ cursor: 'pointer' }}>
|
||||||
{t('delete')}
|
{t('delete')}
|
||||||
</a>
|
</a>
|
||||||
{error && <span className="red-info2"> {error}</span>}
|
{error && <span className="red-info2"> {error}</span>}
|
||||||
|
|||||||
@@ -1,8 +1,8 @@
|
|||||||
'use client'
|
'use client'
|
||||||
|
|
||||||
import { useState } from 'react'
|
import { useState } from 'react'
|
||||||
|
import { useLocale } from 'next-intl'
|
||||||
import { Link } from '@/i18n/navigation'
|
import { Link } from '@/i18n/navigation'
|
||||||
import { useRouter } from '@/i18n/navigation'
|
|
||||||
|
|
||||||
const SERVER_NAME = 'NovaWoW'
|
const SERVER_NAME = 'NovaWoW'
|
||||||
|
|
||||||
@@ -26,28 +26,19 @@ export function SiteHeader({
|
|||||||
realmHref: string
|
realmHref: string
|
||||||
playersHref: string
|
playersHref: string
|
||||||
}) {
|
}) {
|
||||||
const router = useRouter()
|
const locale = useLocale()
|
||||||
const [open, setOpen] = useState(false)
|
const [open, setOpen] = useState(false)
|
||||||
|
|
||||||
async function logout(e: React.MouseEvent) {
|
|
||||||
e.preventDefault()
|
|
||||||
await fetch('/api/auth/logout', { method: 'POST', credentials: 'same-origin' })
|
|
||||||
router.push('/')
|
|
||||||
router.refresh()
|
|
||||||
}
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<header>
|
<header>
|
||||||
<div className="nav-wrapper">
|
<div className="nav-wrapper">
|
||||||
<div className={open ? 'nav-bar responsive' : 'nav-bar'} id="nw-nav-bar">
|
<div className={open ? 'nav-bar responsive' : 'nav-bar'} id="nw-nav-bar">
|
||||||
<a
|
<a
|
||||||
href="javascript:void(0);"
|
role="button"
|
||||||
|
tabIndex={0}
|
||||||
className={open ? 'icon change' : 'icon'}
|
className={open ? 'icon change' : 'icon'}
|
||||||
id="menu-icon"
|
id="menu-icon"
|
||||||
onClick={(e) => {
|
onClick={() => setOpen((v) => !v)}
|
||||||
e.preventDefault()
|
|
||||||
setOpen((v) => !v)
|
|
||||||
}}
|
|
||||||
>
|
>
|
||||||
<div className="bar1"></div>
|
<div className="bar1"></div>
|
||||||
<div className="bar2"></div>
|
<div className="bar2"></div>
|
||||||
@@ -129,9 +120,7 @@ export function SiteHeader({
|
|||||||
<Link href="/my-account">MI CUENTA</Link>
|
<Link href="/my-account">MI CUENTA</Link>
|
||||||
</p>
|
</p>
|
||||||
<p>
|
<p>
|
||||||
<a href="javascript:void(0);" onClick={logout}>
|
<a href={`/${locale}/log-out`}>DESCONECTAR</a>
|
||||||
DESCONECTAR
|
|
||||||
</a>
|
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
@@ -152,7 +141,7 @@ export function SiteHeader({
|
|||||||
title={SERVER_NAME}
|
title={SERVER_NAME}
|
||||||
/>
|
/>
|
||||||
</a>
|
</a>
|
||||||
<a href="javascript:void(0);">
|
<a href={`/${locale}`}>
|
||||||
<img
|
<img
|
||||||
className="nw-logo"
|
className="nw-logo"
|
||||||
src="/nw-themes/nw-ryu/nw-images/nw-logos/nw-logo.webp"
|
src="/nw-themes/nw-ryu/nw-images/nw-logos/nw-logo.webp"
|
||||||
|
|||||||
Reference in New Issue
Block a user