Skip to content

Align get started section headers horizontally#1047

Merged
davidpgero merged 9 commits intorubyforgood:mainfrom
r-spell:align-get-started-section-headers-horizontally
Feb 25, 2026
Merged

Align get started section headers horizontally#1047
davidpgero merged 9 commits intorubyforgood:mainfrom
r-spell:align-get-started-section-headers-horizontally

Conversation

@r-spell
Copy link
Collaborator

@r-spell r-spell commented Feb 24, 2026

Pull Request

Summary

Changes section header images and styling.

Related Issue

Resolves
#919

Changes

  • Adds new number icon images (I got the pngs out of the Figma, and tweaked the canvas size and resolution, if you have other preferred versions of the images please let me know)
  • Removes the old number icon images.
  • Tweaks the flex, column, and style settings to better match the sizes and alignments in the Figma prototype

Screenshots

Screenshot 2026-02-17 at 9 41 37 AM

Checklist

  • Issue is assigned (commenting on the issue page is needed)
  • Issue link added to the PR's description
  • Branch created from main
  • Commits are small and descriptive (🤞)
  • Ran linter and fixed issues
  • Ran tests and all tests pass
  • CI checks passing
  • Review requested from team members

Notes

  • This is my first time making a PR to this project.

@r-spell r-spell requested a review from davidpgero February 24, 2026 19:13
@r-spell r-spell marked this pull request as ready for review February 24, 2026 19:13
@r-spell r-spell self-assigned this Feb 24, 2026
@r-spell r-spell requested a review from pollygee February 24, 2026 20:48
@r-spell r-spell linked an issue Feb 24, 2026 that may be closed by this pull request
Copy link
Collaborator

@davidpgero davidpgero left a comment

Choose a reason for hiding this comment

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

pretty cool! I like that you added image_tag "#{s[:num]}_Number.png" convention here 👍

@davidpgero davidpgero merged commit 9ca2499 into rubyforgood:main Feb 25, 2026
2 checks passed
@r-spell
Copy link
Collaborator Author

r-spell commented Feb 25, 2026

pretty cool! I like that you added image_tag "#{s[:num]}_Number.png" convention here 👍

@davidpgero thank you for the fast turnaround on the review. I'm so excited to have been able to contribute.

@r-spell r-spell deleted the align-get-started-section-headers-horizontally branch February 25, 2026 13:13
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.

Issue: Get Started section headers should be aligned horizontally››

2 participants