Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @src/components/layout/Navbar.tsx at line 109, The mobile GitHub CTA
currently nests a native inside an because Button (from
src/components/ui/Button) defaults asChild=false; update the CTA so Button
receives the prop asChild to render the anchor element instead of a nested
button—i.e., change the Button usage in the mobile CTA to <Button asChild ...>
(preserving existing className and props) so the anchor becomes the rendered
element and the invalid nesting is removed.
Originally posted by @jaseel0 in #154 (comment)
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
@src/components/layout/Navbar.tsxat line 109, The mobile GitHub CTAcurrently nests a native inside an because Button (from
src/components/ui/Button) defaults asChild=false; update the CTA so Button
receives the prop asChild to render the anchor element instead of a nested
button—i.e., change the Button usage in the mobile CTA to <Button asChild ...>
(preserving existing className and props) so the anchor becomes the rendered
element and the invalid nesting is removed.
Originally posted by @jaseel0 in #154 (comment)