Skip to content

Update style.css#62

Open
Vinit3110 wants to merge 1 commit intoGDGC-MET:mainfrom
Vinit3110:Vinit3110-patch-4
Open

Update style.css#62
Vinit3110 wants to merge 1 commit intoGDGC-MET:mainfrom
Vinit3110:Vinit3110-patch-4

Conversation

@Vinit3110
Copy link

There were basic syntax errors here and there.

  1. Gradients and CSS variables
  • Fixed: --gradient custom property (removed extra comma/parenthesis).
  • Fixed: .skills-bg gradient definition (closed missing parenthesis).
  1. Media queries syntax
  • Removed: malformed @media rules like @media "max-width: 1024px"{ and @media ('max-width: 480px").
  • Added: valid @media (max-width: 1024px | 768px | 480px) blocks so responsive styles now apply correctly.
  1. Loader and stacking contexts
  • Removed: conflicting/duplicate z-index declarations on .loading-screen.
  • Added: a single, consistent z-index to ensure loader sits above content while active.
  • Fixed: .loading-ring animation declaration (removed the conflicting 0s animation; kept the intended spin).
  1. Transitions, opacity, and hover states
  • Removed: transition: none on .social-links a (restores smooth hover).
  • Fixed: inconsistent opacity declarations (e.g., opacity: 90 replaced with standard 0/1 sequence).
  • Added: predictable hover-reveal for overlays (e.g., .project-overlay starts at opacity: 0; transitions to visible on hover).
  1. Visual consistency and overrides
  • Removed: .skill-icon { color: red !important; } which forced incorrect icon color across the theme.
  • Fixed: duplicate/contradictory transform-origin on pseudo-elements (now a single, correct value).
  • Adjusted: hero title spacing (replaced awkward negative margins with standard margins for better layout stability).
  • Class name alignment and cleanup
    • Removed: orphaned/typoed class selectors (e.g., .skill-tagss that didn’t match the HTML).
    • Added: or aligned .skill-tags / .skill-tag rules to match HTML classes consistently across breakpoints.
Screenshot (222) Screenshot (224) image

The Above changes were made using AI.

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.

1 participant