Skip to content

feat(*): responsive footer for small screens - I133#190

Merged
DianaLease merged 4 commits intoaccordproject:mainfrom
teja-pola:teja/133/responsive-footer-for-small-breakpoints
Mar 7, 2025
Merged

feat(*): responsive footer for small screens - I133#190
DianaLease merged 4 commits intoaccordproject:mainfrom
teja-pola:teja/133/responsive-footer-for-small-breakpoints

Conversation

@teja-pola
Copy link
Contributor

Closes #133

Summary

-This PR improves the responsiveness and appearance of the footer to ensure a better user experience on different screen sizes.

Changes

-- Made the footer collapsible on smaller screens with a visible "Expand Footer" button.

  • Ensured the footer sections are properly aligned in full-screen mode (left and right positioning).

Flags

-none

Screenshots or Video

Before

Screenshot 2025-03-07 094053

After

Untitled.video.-.Made.with.Clipchamp.mp4

Related Issues

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Vital features and changes captured in unit and/or integration tests
  • Commits messages follow AP format
  • Extend the documentation, if necessary
  • Merging to main from fork:branchname

Signed-off-by: Dharma Teja <dteja2468@gmail.com>
@teja-pola teja-pola requested a review from a team as a code owner March 7, 2025 04:24
@netlify
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for ap-template-playground ready!

Name Link
🔨 Latest commit ca66d0e
🔍 Latest deploy log https://app.netlify.com/sites/ap-template-playground/deploys/67cb36b43ef9880008ffc163
😎 Deploy Preview https://deploy-preview-190--ap-template-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

One comment. Otherwise looks good!

src/App.tsx Outdated
Best viewed on desktop

</div>
)}
Copy link
Member

Choose a reason for hiding this comment

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

This whole section (lines 189-200) can now be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, silly me, that's done now. please check.

Signed-off-by: Dharma Teja <dteja2468@gmail.com>
@teja-pola teja-pola force-pushed the teja/133/responsive-footer-for-small-breakpoints branch from dc34978 to 59f54e5 Compare March 7, 2025 16:50
Signed-off-by: Dharma Teja <dteja2468@gmail.com>
Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: Dharma Teja <dteja2468@gmail.com>
@teja-pola
Copy link
Contributor Author

Great, all tests passed. @DianaLease Can you please merge if everything is alright.

@DianaLease DianaLease merged commit 7ae2812 into accordproject:main Mar 7, 2025
7 checks passed
@DianaLease
Copy link
Member

Great, all tests passed. @DianaLease Can you please merge if everything is alright.

Merged, thank you for your contribution!

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.

Enhance Footer Responsiveness for Small Breakpoints

2 participants