Replace onclick navigation with proper links
Converts cancel buttons that use onclick="window.location.href='...'" to proper A tags with href. This improves accessibility (keyboard navigation, right-click options) and semantics while maintaining the same button styling. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -4,7 +4,7 @@
|
|||||||
from collections.abc import Callable
|
from collections.abc import Callable
|
||||||
from typing import Any
|
from typing import Any
|
||||||
|
|
||||||
from fasthtml.common import H2, H3, Div, Form, Hidden, Input, Option, P, Script, Select, Span
|
from fasthtml.common import H2, H3, A, Div, Form, Hidden, Input, Option, P, Script, Select, Span
|
||||||
from monsterui.all import (
|
from monsterui.all import (
|
||||||
Alert,
|
Alert,
|
||||||
AlertT,
|
AlertT,
|
||||||
@@ -86,11 +86,10 @@ def diff_confirmation_panel(
|
|||||||
Hidden(name="confirmed", value="true"),
|
Hidden(name="confirmed", value="true"),
|
||||||
Hidden(name="nonce", value=str(ULID())),
|
Hidden(name="nonce", value=str(ULID())),
|
||||||
Div(
|
Div(
|
||||||
Button(
|
A(
|
||||||
"Cancel",
|
"Cancel",
|
||||||
type="button",
|
href=cancel_url,
|
||||||
cls=ButtonT.default,
|
cls=ButtonT.default,
|
||||||
onclick=f"window.location.href='{cancel_url}'",
|
|
||||||
),
|
),
|
||||||
Button(
|
Button(
|
||||||
confirm_button_text,
|
confirm_button_text,
|
||||||
|
|||||||
@@ -4,7 +4,7 @@
|
|||||||
from collections.abc import Callable
|
from collections.abc import Callable
|
||||||
from typing import Any
|
from typing import Any
|
||||||
|
|
||||||
from fasthtml.common import H2, Div, Form, Hidden, Option, P, Select, Span
|
from fasthtml.common import H2, A, Div, Form, Hidden, Option, P, Select, Span
|
||||||
from monsterui.all import Alert, AlertT, Button, ButtonT, FormLabel, LabelInput, LabelTextArea
|
from monsterui.all import Alert, AlertT, Button, ButtonT, FormLabel, LabelInput, LabelTextArea
|
||||||
from ulid import ULID
|
from ulid import ULID
|
||||||
|
|
||||||
@@ -253,11 +253,10 @@ def diff_panel(
|
|||||||
Hidden(name="confirmed", value="true"),
|
Hidden(name="confirmed", value="true"),
|
||||||
Hidden(name="nonce", value=str(ULID())),
|
Hidden(name="nonce", value=str(ULID())),
|
||||||
Div(
|
Div(
|
||||||
Button(
|
A(
|
||||||
"Cancel",
|
"Cancel",
|
||||||
type="button",
|
href="/move",
|
||||||
cls=ButtonT.default,
|
cls=ButtonT.default,
|
||||||
onclick="window.location.href='/move'",
|
|
||||||
),
|
),
|
||||||
Button(
|
Button(
|
||||||
f"Confirm Move ({diff.server_count} animals)",
|
f"Confirm Move ({diff.server_count} animals)",
|
||||||
|
|||||||
Reference in New Issue
Block a user