Skip to content

Conversation

@cherylpinto
Copy link
Contributor

@cherylpinto cherylpinto commented Oct 5, 2025


name: Pull Request
about: Submit changes to the Sugar Labs website for review

📝 Description

This PR adds a fully designed, dark-mode-compatible Contact Us page to the Sugar Labs website.
The page includes:

  • Introductory section with animated heading, description, and illustration.
  • Contact cards for Mail, Phone, and Email.
  • "Connect With Our Community" section displaying all social media links.
  • A Matrix Chat call-to-action card.
  • Smooth and optimized animations using Framer Motion.
  • Responsive design with consistent Header, Footer, and socialLinks integration.
  • Full support for dark mode

This enhances usability, accessibility, and visual consistency across light and dark themes.

🔗 Related Issue

Fixes #446

🔄 Type of Change

  • 📱 New Feature (new page, component, or functionality)
  • 🎨 UI/UX Update (visual changes, styling improvements)
  • 🌙 Dark Mode Compatibility
  • 🐛 Bug Fix
  • ♿ Accessibility Enhancement
  • 🔒 Security Update
  • 📦 Dependency Update
  • 🧹 Code Refactoring
  • 🧪 Test Updates

📷 Visual Changes

Screenshots

💻 Desktop – Light Mode

image image

🌙 Desktop – Dark Mode

image image

📱 Mobile View

image image image image

🧪 Testing Performed

📱 Browser Compatibility

  • Chrome (Version: 140+)
  • Firefox (Latest)
  • Safari (Latest)
  • Edge (Latest)
  • Mobile Chrome (Android)
  • Mobile Safari (iOS)

🖥️ Responsive Design

  • Desktop (1200px+)
  • Tablet (768px - 1199px)
  • Mobile (320px - 767px)

✅ Test Cases

  1. Verified Mail, Phone, and Email cards render correctly and links function.
  2. Confirmed all social links open in new tabs.
  3. Checked Matrix chat button navigation.
  4. Tested light/dark mode transitions and color contrast.
  5. Ensured animations perform smoothly on all screen sizes.

♿ Accessibility

  • Proper heading hierarchy maintained
  • Alt text for all icons/images
  • Dark mode colors meet contrast ratios
  • Keyboard navigation works correctly
  • Screen reader testing performed

📋 PR Checklist

  • My code follows the project's coding style guidelines
  • I have tested these changes locally
  • I have verified dark mode compatibility
  • My changes generate no new warnings or console errors
  • All existing tests pass successfully
  • I have validated all links and icons

💭 Additional Notes

  • Open to feedback

📚 Reviewer Resources

Thank you for reviewing this PR! 🎉

@github-actions
Copy link

github-actions bot commented Oct 5, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@cherylpinto cherylpinto changed the title Feat/issue 446 Contact-Us UI Oct 5, 2025
@cherylpinto
Copy link
Contributor Author

Hi @sa-fw-an can you review my work.

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 5, 2025

@cherylpinto It would be better if you attach screenshots from next time :)

@cherylpinto
Copy link
Contributor Author

cherylpinto commented Oct 5, 2025

@sa-fw-an they are there in the pr under visual changes

Its a dropdown under VISUAL CHANGES HEADING

If you want, I can send them again!

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 5, 2025

@sa-fw-an they are there in the pr under visual changes

its a dropdown

Ah, My bad.. Looks Great by the way, it would be better if the Text is Red in color instead of Blue I guess.

@cherylpinto
Copy link
Contributor Author

Okay will do it, and update the PR.

@cherylpinto
Copy link
Contributor Author

cherylpinto commented Oct 5, 2025

So just wanted to confirm you want me to change the entire theme of this page from blue to red? Like the Leadership page? @sa-fw-an

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 5, 2025

So just wanted to confirm you want me to change the entire theme of this page from blue to red? Like the Leadership page?

Yes, I guess that would look good. Not sure how it would look tho, I suggest attach screenshots of how it would look if its Red in Color, if it looks good you can push the change, else keep the Blue theme itself.

@cherylpinto
Copy link
Contributor Author

Hi @sa-fw-an , I changed the theme from blue to red. Attached are the screenshots of the website. Kindly Review the same, so that I can update the PR accordingly

Light Mode:
image
image

Dark Mode:
image

image

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 6, 2025

@cherylpinto Looks good, Please push the code so that I can merge it :)

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 6, 2025

Can you try making these svgs more consistent in the color scheme. Here the linkedin svg is colored and the rest are in black.
Screenshot 2025-10-06 at 8 44 54 PM

Please try making them consistent by just rendering those svgs in black color like in the image below:
Screenshot 2025-10-06 at 8 45 02 PM

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

❌ Checks Failed

Status: 🚫 Not ready to merge

Please fix the following issues before merging:

📝 Code Linting Failed

Issue: Code formatting or style violations detected.

Specific problems:

• Code formatting issues detected

How to fix:

Platform Command Description
🐧 Unix/macOS/Linux npm run format Auto-fix all formatting issues
🪟 Windows npm run format:file <filename> Fix specific files
🔍 Check Only npm run format:check Check formatting without fixing

Need help with linting? Check out the Linting Guide for Windows Users for detailed instructions.


🛠️ Next Steps

  1. Fix the issues mentioned above
  2. Test locally to ensure everything works
  3. Push your fixes to this branch
  4. Wait for re-check - This bot will automatically run again

🤖 This comment will be updated automatically when you push new commits

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@cherylpinto
Copy link
Contributor Author

Yes i have done it
image

that happened since someone just changed the linkedin svg(that was blue) and since I pulled the changes, it took that svg.
I have updated it.

@sa-fw-an
Copy link
Member

sa-fw-an commented Oct 6, 2025

Great work @cherylpinto , Looks Good :)

@sa-fw-an sa-fw-an merged commit 8d810e3 into sugarlabs:main Oct 6, 2025
3 checks passed
@sa-fw-an sa-fw-an mentioned this pull request Oct 6, 2025
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.

Redesign Contact Us Page to Match Website Theme

2 participants