Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Shop category section UI/interaction by simplifying hover-driven behavior and adding a persistent section title.
Changes:
- Removed hover state management and the hover-based scroll speed change for the background title animation.
- Added a static section title pinned to the top of the category card.
- Made product cards always visible on desktop (removed hover/opacity gating).
Comments suppressed due to low confidence (1)
src/components/Shop/CategorySection.tsx:146
- The background scrolling title appears to be decorative, but it’s rendered as repeated
text without being hidden from assistive tech. This can cause screen readers to announce the title many times (and now there’s also a visible title at the top). Mark this decorative block as aria-hidden (or otherwise exclude it from the accessibility tree).
{/* Background Title - Only render animation when in view */}
{isInView && (
<motion.div
ref={scrollRef}
className={`absolute inset-0 flex ${orientation === "vertical" ? "flex-col justify-center" : "items-center"} overflow-hidden pointer-events-none group-hover:opacity-10 transition-opacity duration-300 ease-in-out will-change-[opacity]`}
initial={{ opacity: 0 }}
animate={{ opacity: 0.3 }}
exit={{ opacity: 0 }}
transition={{ delay: 0.6, duration: 0.5 }}
style={
{
transform: "translateZ(0)",
backfaceVisibility: "hidden" as const,
} as React.CSSProperties
}
>
<div
className={`${getScrollClass()} whitespace-nowrap flex ${orientation === "vertical" ? "flex-col" : ""}`}
style={{
transform: "translateZ(0)",
willChange: "transform",
}}
>
<h2 className={getTextClass()}>
{title.toUpperCase()} {title.toUpperCase()} {title.toUpperCase()}{" "}
{title.toUpperCase()} {title.toUpperCase()} {title.toUpperCase()}
</h2>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Ben Juntilla <ben@benjuntilla.com>
benjuntilla
approved these changes
Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.