Skip to content

feat(site): convert description prop on PageSection to named slot - #55

Merged
suejinkim20 merged 3 commits into
stagecc:mainfrom
suejinkim20:feature/page-section-description-slot
Aug 26, 2026
Merged

feat(site): convert description prop on PageSection to named slot#55
suejinkim20 merged 3 commits into
stagecc:mainfrom
suejinkim20:feature/page-section-description-slot

Conversation

@suejinkim20

@suejinkim20 suejinkim20 commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

This PR converts the "description" prop on the PageSection component to a named slot.

@suejinkim20
suejinkim20 force-pushed the feature/page-section-description-slot branch 2 times, most recently from 94f4eb1 to d415821 Compare August 25, 2026 20:47
@suejinkim20
suejinkim20 force-pushed the feature/page-section-description-slot branch from d415821 to 4d97812 Compare August 25, 2026 20:49
@suejinkim20
suejinkim20 requested a review from mbwatson August 25, 2026 20:59
@suejinkim20 suejinkim20 changed the title Feature/page section description slot feat(site): convert description prop on PageSection to named slot Aug 25, 2026

@mbwatson mbwatson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good!

because PageSection will now use a named description slot, let's update the packages/ui-astro/README.md note about receiving "all content via props" to mention props + slots; something like:

Components are presentational and receive content through typed props plus default/named slots for richer markup.

@suejinkim20
suejinkim20 merged commit b45bb2a into stagecc:main Aug 26, 2026
5 checks passed
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.

2 participants