chore: improve footer, theming, and fix CSS/JS errors (AI-assisted)#72
Open
bhagyashrishewale2005 wants to merge 1 commit intoGDGC-MET:mainfrom
Open
chore: improve footer, theming, and fix CSS/JS errors (AI-assisted)#72bhagyashrishewale2005 wants to merge 1 commit intoGDGC-MET:mainfrom
bhagyashrishewale2005 wants to merge 1 commit intoGDGC-MET:mainfrom
Conversation
Files changed: index.html, style.css, script.js\n\n- Responsive semantic footer\n- Centralized theming with CSS variables + dark/purple themes\n- Fixed CSS syntax issues and removed debug rules\n- Added header theme-toggle UI and JS persistence\n- Fixed JS mousemove bug and cleaned debug timers\n\nChanges applied with AI assistance by bhagyashrishewale2005
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.
Files touched: index.html, style.css, script.js.
index.html
Replaced the initial bare footer with a semantic, responsive footer layout:
Three-column grid: About, Quick Links, Contact + social icons.
Added clearer microcopy and max-width constraints for readability.
Ensured footer links and social icons are keyboard-focusable and include accessible labels where appropriate.
Added a compact theme-toggle control to the header (three buttons: Default / Dark / Purple) so users can switch site palettes.
Fixed several small HTML markup issues in header/CTA areas (removed stray tags and duplicate attributes).
style.css
Introduced a centralized theming system using CSS custom properties in :root:
Default "simple, rich" palette plus :root[data-theme="purple"] and :root[data-theme="dark"] overrides.
Added RGB component variables (e.g., --primary-rgb) so rgba() can be derived from variables.
Applied the chosen pastel palette (Lavender Peach) to the variable set.
Replaced hard-coded color/rgb usages (notably scattered blue rgba values) with theme variables (e.g., rgba(var(--primary-rgb), 0.12) or var(--gradient)).
Implemented footer styling:
.footer-grid responsive grid rules, .footer-bottom copyright/muted text.
Focus and hover states for links and social icons for keyboard/visual affordances.
Mobile stacking rules for small screens.
Added styles for the header theme toggle (.theme-toggle, .theme-btn) with hover/focus states and accessible sizing.
Fixed multiple CSS syntax and quality problems found in the initial file:
Corrected malformed gradients and missing/extra punctuation.
Fixed media query syntax and duplicate/conflicting declarations.
Standardized background-clip usage and other cross-browser niceties.
Cleaned stray debug/test CSS (e.g., removed or corrected accidental !important rules and red debug color definitions).
Kept responsive rules and reduced-motion support.
script.js
Fixed a broken mousemove/cursor assignment that used invalid template-like syntax; corrected to valid DOM assignments.
Removed leftover debug intervals/logging that could cause performance/noise.
Implemented theme application and persistence:
applyTheme(name) sets/removes data-theme on document.documentElement.
Theme buttons wired to call applyTheme(...) and persist selection in localStorage (key: site-theme).
Initialization reads persisted theme and applies it on load.
Minor JavaScript cleanup: removed duplicate DOMContentLoaded listeners and tightened event handlers to avoid leaks.
Validation & notes
Performed quick parse/lint-like checks after edits and resolved reported CSS/JS parse issues.
Left functional behavior intact (animations, layout, particles). Suggested next step: wire particle colors to CSS variables so they follow active theme.
No build system changes required — edits are limited to the static files.
Attribution
These changes were performed with the assistance of an AI coding assistant; changes were reviewed and applied to the project files.














Files changed: index.html, style.css, script.js\n\n- Responsive semantic footer\n- Centralized theming with CSS variables + dark/purple themes\n- Fixed CSS syntax issues and removed debug rules\n- Added header theme-toggle UI and JS persistence\n- Fixed JS mousemove bug and cleaned debug timers\n\nChanges applied with AI assistance by bhagyashrishewale2005