Skip to content

Update index.html - #60

Open
Vinit3110 wants to merge 1 commit into
GDGC-MET:mainfrom
Vinit3110:Vinit3110-patch-2
Open

Update index.html#60
Vinit3110 wants to merge 1 commit into
GDGC-MET:mainfrom
Vinit3110:Vinit3110-patch-2

Conversation

@Vinit3110

Copy link
Copy Markdown
  1. Navigation links and buttons
  • Removed: duplicate onclick handlers on the same link; duplicate target attributes (e.g., target="_self" and target="_blank" together).
  • Added: a single onclick="scrollToSection('...')" per nav item.
  • Fixed: misspelled/malformed ARIA attributes (aria-lable/arialabel → aria-label).
  1. Hero CTA and SVG markup
  • Removed: stray closing tag and other mis-nested/extra closing tags that broke the button and SVG structure.
  • Added: proper closing and balanced button markup (open/close order corrected).
  • About “stats” layout
    • Removed: premature closing of the about-stats container that left .stat items outside their wrapper.
    • Added: .stat items nested correctly inside .about-stats (restoring intended grid/flex behavior).
  1. Skills section identity
  • Removed: duplicate id attribute (id="skills" and id="technical-skills" simultaneously).
  • Kept/Added: single id="skills" for valid DOM and anchor-targeting.
  1. Contact section (links and form)
  • Fixed: malformed mailto link attributes (e.g., broken quotes on title/aria-label).
  • Removed: duplicated required attributes on inputs.
  • Removed: disabled="disabled" on the submit button (form is now actually submit-able).
  1. Accessibility and semantics
  • Added: consistent aria-labels on interactive elements (logo, social icons, CTA buttons).
  • Removed: invalid/empty attributes that caused validation warnings and potential JS selection issues.
image image

The above changes were made using the help of AI for identifying and fixing the major issue.

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