Skip to content

Conversation

@hypha
Copy link
Contributor

@hypha hypha commented Feb 5, 2025

While working on the accordion styling for session types in PR #989, it became clear that extracting it into a reusable component would make things easier for the team.

@hypha hypha requested a review from egeakman February 5, 2025 18:50
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Preview available

Key Value
url https://component-accordion.ep-preview.click
last update 2025-03-10T14:44:09.319Z

@egeakman
Copy link
Member

egeakman commented Feb 7, 2025

Thanks for the PR @hypha ❤️

This works but I would prefer using Astro wherever we can. Could we use Astro if it's not too much work?
We could use this for now and I could do the Astro migration at a later time too.

What do you think?

@hypha
Copy link
Contributor Author

hypha commented Feb 7, 2025

@egeakman sure! Since there is no rush for this, I'll try next week! Good point! <3

@hypha
Copy link
Contributor Author

hypha commented Feb 13, 2025

@egeakman I've converted it to an astro component (I hope!) Would you mind reviewing it whenever you got the time? <3

@egeakman
Copy link
Member

egeakman commented Mar 9, 2025

I'd like to merge #1024 first, so we can test this! Thank you Raquel ❤️

@hypha
Copy link
Contributor Author

hypha commented Mar 9, 2025

I'd like to merge #1024 first, so we can test this! Thank you Raquel ❤️

Exactly what I thought too! I will definitely add the use case to test page once it is merged! <3

@egeakman egeakman linked an issue Mar 10, 2025 that may be closed by this pull request
@hypha
Copy link
Contributor Author

hypha commented Mar 10, 2025

@egeakman, accordion example added in ca4cf01 and ready to be reviewed for reals now 😊

Copy link
Member

@egeakman egeakman left a comment

Choose a reason for hiding this comment

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

Amazing! Thank you ❤️

@egeakman egeakman merged commit 6f3e5e0 into ep2025 Mar 10, 2025
2 checks passed
@egeakman egeakman deleted the component-accordion branch March 10, 2025 14:46
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.

Accordion Component Needed

3 participants