Skip to content

Conversation

@KrishLalakiya
Copy link
Contributor

Description

Added Parent div and assign hover to it so if you are initially hovering the card then animation will continue, only go back when you leave that parent div. So no more stutter in animation.

Before Fix

Screen.Recording.2025-10-29.025046.mp4

After Fix

Screen.Recording.2025-10-29.030331.mp4

Description:
This PR fixes several mobile layout bugs and improves site usability.

Changes Made:

Responsive Header:

Added display: flex to the desktop header to correctly align the logo and nav links.

Implemented a CSS-only "hamburger menu" for mobile (using the "checkbox hack"). The menu is now functional and animates open and closed.

Centered Gallery:

Added align-items: center to the .card-grid media query. This now centers the cards properly when they stack on mobile.

Accessibility Fix:

Corrected the low-contrast text. The default body text is now dark, and color: #fff is only used on sections with dark backgrounds (like the header and footer).
Introduced a new .card-wrapper div to encapsulate the card container, improving the structure and separation of the 3D flipping card component. Updated CSS to reflect the new hierarchy and adjusted hover effect to target the container within the wrapper.
@KrishLalakiya KrishLalakiya changed the title Fixes 948 3D Flipping Card Component animation stutter and Closes #948 Fixed 3D Flipping Card Component animation stutter and Closes #948 Oct 28, 2025
Copy link
Member

@aayush105 aayush105 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great fix @KrishLalakiya

Thank you for your contribution.

@aayush105
Copy link
Member

Here like you have made changes in minecraft folder as well, can you please check it once?

@KrishLalakiya
Copy link
Contributor Author

KrishLalakiya commented Oct 30, 2025

I was learning to use git using git app so this happened, Its same code i have submitted for Minecraft website, and i have checked it is working properly. You can take both or just take Flipping Commit. @aayush105

@KrishLalakiya
Copy link
Contributor Author

KrishLalakiya commented Oct 30, 2025

100cc6a This commit is main commit . @aayush105

Changed image source paths in Login/Signin.html from absolute to relative paths to ensure images load correctly regardless of deployment location.
@KrishLalakiya
Copy link
Contributor Author

Got it, Till now i was working at 1 issue at time so i was not creating new breach. so if i work on other file it also show in other code pull request. @aayush105

@aayush105 aayush105 merged commit f790cef into you-dont-need:master Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants