Skip to content
This repository has been archived by the owner on Sep 7, 2024. It is now read-only.

fix:Responsiveness on mobile screens including iPhone5 #753

Closed
wants to merge 1 commit into from
Closed

fix:Responsiveness on mobile screens including iPhone5 #753

wants to merge 1 commit into from

Conversation

diva-21
Copy link

@diva-21 diva-21 commented Aug 24, 2023

Fixes Issue

This PR fixes the https://github.com/WeMakeDevs/wemakedevs/issues/721 issue

closes #issue-number

Changes proposed

Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Add all the screenshots which support your changes
image

Summary by CodeRabbit


  • Refactor: Updated the Footer component to improve responsiveness. The gap between grid items in the footer will now be 4 units on small and larger screens, enhancing the visual layout and user experience across different screen sizes.

@coderabbitai
Copy link

coderabbitai bot commented Aug 24, 2023

Image description CodeRabbit

Walkthrough

The change introduces a responsive design to the Footer component in our application. The gap between grid items now adjusts based on the screen size, enhancing the user experience across different devices.

Changes

File Summary
src/components/layout/Footer.jsx Modified the Footer component to introduce responsive behavior. The gap between grid items will be 4 units on small and larger screens, but not on medium-sized screens.

🐇💻

In the land of code, where the screens vary,

A rabbit hopped in, no longer to tarry.

With a twitch of the nose, and a keystroke so light,

He made the grid responsive, oh what a sight!

Now on every device, big or small,

The footer looks great, it stands tall! 🎉🥕

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Actionable comments generated: 0

Commits Files that changed from the base of the PR and between 9e7265c and a99db9f commits.
Files selected for processing (1)
  • src/components/layout/Footer.jsx (1 hunks)
Files skipped from review due to trivial changes (1)
  • src/components/layout/Footer.jsx

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Tag the bot with @coderabbitai in any review comment to ask it questions. The bot will not respond unless explicitly tagged.
  • A review comment is a comment on a diff or a file and not a comment on the PR itself.
  • You can use the mouse to select multiple lines of code to provide exact context to the bot.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Woah! This is your first Pull Request in our community, Congratulations 🎉 and thank you for your contribution! 🚀 The team will review it on availability 💫. In the meantime join our Discord server and checkout our YouTube channel.

@Chandraprakash-Darji
Copy link
Contributor

@diva-21 this issue is already fixed in #750

@Chandraprakash-Darji
Copy link
Contributor

Thanks for your contribution,
Tip: don't work on the issue until It is assigned to you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants