Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make accordion styling as a component #996

Open
wants to merge 3 commits into
base: ep2025
Choose a base branch
from
Open

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
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Preview available

Key Value
url https://component-accordion.ep-preview.click
last update 2025-02-13T17:35:37.655Z

@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

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