Skip to content

Add responsive footer with dynamic year support#141

Open
deenahrajeswari wants to merge 3 commits into
aditya-ai00:mainfrom
deenahrajeswari:feature/add-responsive-footer
Open

Add responsive footer with dynamic year support#141
deenahrajeswari wants to merge 3 commits into
aditya-ai00:mainfrom
deenahrajeswari:feature/add-responsive-footer

Conversation

@deenahrajeswari

Copy link
Copy Markdown

Overview

This PR adds a simple and responsive footer to the Weather Dashboard while preserving the existing UI structure and styling.

Changes Made

  • Added a minimal footer section to the application
  • Implemented dynamic year rendering using JavaScript
  • Ensured footer compatibility with both light and dark modes
  • Maintained responsiveness across desktop and mobile screen sizes
  • Preserved the existing layout and UI without unnecessary modifications

Issues Fixed

  • Added missing footer section to improve UI completeness
  • Improved footer responsiveness for different screen sizes
  • Ensured footer positioning does not overlap existing content

Approach Taken

  • Added a lightweight footer component with minimal text
  • Used JavaScript new Date().getFullYear() for automatic year updates
  • Applied responsive spacing and typography without modifying the existing design structure
  • Kept the footer styling consistent with the current project theme

Testing Performed

  • Verified footer visibility in both light and dark modes
  • Tested responsiveness on:
    - Desktop view
    - Tablet view
    - Mobile view
  • Verified dynamic year updates correctly
  • Ensured no layout breakage or overlap issues occurred

Close issue #114

Screenshot (295) Screenshot (296)

@vercel

vercel Bot commented May 8, 2026

Copy link
Copy Markdown

@deenahrajeswari is attempting to deploy a commit to the aditya-ai00's projects Team on Vercel.

A member of the Team first needs to authorize it.

@deenahrajeswari

Copy link
Copy Markdown
Author

@aditya-ai00 Kindly review the pr and add label to the pr.

@aditya-ai00

Copy link
Copy Markdown
Owner

Hi @deenahrajeswari,

Thanks for the contribution.

The footer addition and responsive improvements look like a good enhancement for improving overall UI completeness. However, the PR is still showing merge conflicts in:

  • index.html
  • script.js

So I’m unable to proceed with the review yet.

Please:

  • pull the latest changes from the main branch
  • resolve all remaining conflicts completely
  • ensure a fresh preview deployment is available
  • verify that the footer works correctly across desktop/mobile layouts and in both light and dark modes

Also, please keep the scope focused only on the footer-related enhancement for Issue #114.

No additional PR label is required from your side.

Once the PR becomes fully conflict-free and stable, I’ll review it again.

Thanks!

@deenahrajeswari

Copy link
Copy Markdown
Author

@aditya-ai00 Merge conflict is resolved

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