Skip to content

UI/UX: Footer Alignment Issue (#31)#32

Closed
ankitkr104 wants to merge 0 commit intoDjedAlliance:mainfrom
ankitkr104:fix/footer-alignment-issue-31
Closed

UI/UX: Footer Alignment Issue (#31)#32
ankitkr104 wants to merge 0 commit intoDjedAlliance:mainfrom
ankitkr104:fix/footer-alignment-issue-31

Conversation

@ankitkr104
Copy link
Contributor

@ankitkr104 ankitkr104 commented Feb 26, 2026

  • Remove side gaps and ensure full-width coverage
  • Fix uneven spacing between footer sections
  • Align text and icons properly
  • Reduce footer height for better proportions
  • Add consistent padding and gap spacing
  • Implement responsive layout for mobile devices
  • Style social icons with glass effect matching navbar
  • Ensure consistent layout across different screen sizes

Screenshots:


Djed image pr

Changes Made

Fixed uneven spacing between footer sections
Corrected misaligned text and icons
Improved layout consistency across different screen sizes
Ensured better responsiveness and visual structure

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions.
  • If applicable, I have made corresponding changes or additions to the documentation.
  • If applicable, I have made corresponding changes or additions to tests.
  • My changes generate no new warnings or errors.
  • I have joined the Stability Nexus's Discord server and I will share a link to this PR with the project maintainers there.
  • I have read the Contribution Guidelines.
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.

Summary by CodeRabbit

  • Style
    • Enhanced footer styling with improved visual hierarchy and spacing.
    • Added responsive design for mobile devices with optimized footer layout.
    • Refined social media links styling and visual presentation.
    • Improved footer branding section appearance and organization.

@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

📝 Walkthrough

Walkthrough

The PR restructures the Footer component's DOM markup and adds comprehensive footer CSS styling with responsive rules. The footer now uses semantic CSS classes for layout organization and includes responsive design for mobile devices.

Changes

Cohort / File(s) Summary
Footer Styling
src/App.css
Added 99 lines of footer CSS selectors including .footer-container, .footer-content, .footer-copyright, .footer-brand, .footer-social, and associated elements. Includes hover/transition effects, social icon styling, and responsive media query (max-width: 768px) for mobile layout with vertical stacking and element reordering.
Footer Component Structure
src/components/pages/footer/Footer.tsx
Reorganized footer DOM hierarchy: simplified outer structure with footer-container and footer-content wrappers. Moved copyright text to footer-copyright block, created footer-brand section for logo and title, and separated social links into footer-social block. Removed layout-oriented class names in favor of new semantic CSS selectors.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related issues

Suggested reviewers

  • Zahnentferner

Poem

🐰 A footer so fine, with styles divine,
Social links dancing in a responsive design,
Mobile and desktop in harmony aligned,
CSS classes leave the old chaos behind! 🦶✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'UI/UX: Footer Alignment Issue (#31)' directly addresses the main purpose of the PR, which is to fix footer alignment, spacing, and responsiveness issues as detailed in the objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/App.css`:
- Line 327: Remove the quotes around the Montserrat family name in the
font-family declaration to satisfy the stylelint rule; locate the font-family
property (font-family: 'Montserrat', sans-serif;) in src/App.css and change it
to use an unquoted family name (Montserrat) so the declaration reads
font-family: Montserrat, sans-serif;.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7da6246 and d5e8df5.

📒 Files selected for processing (2)
  • src/App.css
  • src/components/pages/footer/Footer.tsx

src/App.css Outdated
}

.footer-copyright {
font-family: 'Montserrat', sans-serif;
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix Stylelint violation for font-family on Line 327.

"Montserrat" is quoted and triggers font-family-name-quotes. Use the unquoted family name here.

Proposed fix
-.footer-copyright {
-  font-family: 'Montserrat', sans-serif;
+.footer-copyright {
+  font-family: Montserrat, sans-serif;
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
font-family: 'Montserrat', sans-serif;
font-family: Montserrat, sans-serif;
🧰 Tools
🪛 Stylelint (17.3.0)

[error] 327-327: Unexpected quotes around "Montserrat" (font-family-name-quotes)

(font-family-name-quotes)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/App.css` at line 327, Remove the quotes around the Montserrat family name
in the font-family declaration to satisfy the stylelint rule; locate the
font-family property (font-family: 'Montserrat', sans-serif;) in src/App.css and
change it to use an unquoted family name (Montserrat) so the declaration reads
font-family: Montserrat, sans-serif;.

@ankitkr104 ankitkr104 closed this Feb 26, 2026
@ankitkr104 ankitkr104 force-pushed the fix/footer-alignment-issue-31 branch from d5e8df5 to 7da6246 Compare February 26, 2026 08:03
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