Skip to content

Conversation

@Dakshsharma1108
Copy link
Contributor

@Dakshsharma1108 Dakshsharma1108 commented Nov 30, 2025

📝 Description

  1. Added complete steps and images
  • Expanded the guide with detailed step-by-step content.
  • Included the correct images for each step to improve clarity and usability.
  1. Removed unused fields from raspberryLogoCardsData
  • Removed description and button-related fields because the Raspberry guide no longer uses them.
  • Keeps the data cleaner.
  1. Converted ScrollToSteps into a reusable useScroll hook
  • Created a new custom hook (useScroll) that handles scrolling to step sections.
  • This makes the scrolling behavior reusable across other pages and better aligned with React patterns.
  1. Fixed step-switching bug when changing guides
  • Previously, switching guides while on a later step could leave the UI blank if the newly selected guide had fewer steps.
  • Fixed by resetting the steps UI when the guide changes — the StepsToUse component is remounted using a [key]so its internal [currentStep] resets safely.

🔗 Related Issue

Fixes #444

🔄 Type of Change

  • 📱 New Feature (new page, component, or functionality)
  • 🎨 UI/UX Update (visual changes, styling improvements)
  • 📖 Content Update (text changes, documentation)
  • 🐛 Bug Fix

📷 Visual Changes

Visit following url for live preview -
https://www-v2-mac-guide.vercel.app/bootablesoas

🖥️ Responsive Design

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

@github-actions
Copy link

❌ 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:

• TypeScript linting failed

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

❌ 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:

• TypeScript linting failed

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

🎉 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 Dec 1, 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. 👏

@Dakshsharma1108
Copy link
Contributor Author

please review @sa-fw-an

@sa-fw-an sa-fw-an merged commit 0e4a71d into sugarlabs:main Dec 1, 2025
3 checks passed
@Dakshsharma1108 Dakshsharma1108 deleted the feature/guide-for-mac branch December 1, 2025 16:20
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.

Visual Guide to Booting Sugar on a Stick for MAC and Windows

2 participants