Skip to content

Conversation

@kartikktripathi
Copy link

@kartikktripathi kartikktripathi commented Nov 9, 2025


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

📝 Description

Added some space between the logo and the text for enhancement.
Removed unnecessary space between the commas and the words.

🔗 Related Issue

Fixes #534

🔄 Type of Change

  • 🎨 UI/UX Update (visual changes, styling improvements)
  • 🐛 Bug Fix

📷 Visual Changes

Screenshots / GIFs Screenshot 2025-11-09 at 4 47 17 PM

🧪 Testing Performed

📱 Browser Compatibility

  • Chrome (Version: )
  • Firefox (Version: )
  • Safari (Version: )
  • Edge (Version: )
  • Mobile Chrome (Device: )
  • Mobile Safari (Device: )

🖥️ Responsive Design

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

✅ Test Cases

  1. The Space provided between the logo and the text increases the cleanliness of the page.
  2. The page works well in both colour modes.

♿ Accessibility

  • Proper heading hierarchy maintained
  • ARIA labels added where needed
  • Color contrast requirements met
  • 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 updated the documentation accordingly
  • My changes generate no new warnings or console errors
  • I have added tests that prove my fix/feature works
  • All existing tests pass successfully
  • I have checked for and resolved any merge conflicts
  • I have optimized images/assets (if applicable)
  • I have validated all links are working correctly

💭 Additional Notes

This PR includes some basic updates to the page's Frontend, and the small contributions matter as we work to make it a better work piece.

📚 Reviewer Resources

Thank you for contributing to the Sugar Labs website! 🎉

@github-actions
Copy link

github-actions bot commented Nov 9, 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. 👏

@kartikktripathi
Copy link
Author

Hey @sa-fw-an, I fixed some spacing issues. Could you please check? Thanks!

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.

[Bug]: <p> tag items dislocated in some dimensions

1 participant