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

astro-tabs numbering is messed up in MDX #64

Open
fviolette opened this issue Nov 28, 2022 · 2 comments
Open

astro-tabs numbering is messed up in MDX #64

fviolette opened this issue Nov 28, 2022 · 2 comments
Labels
bug-fix Something isn't working good first issue Good for newcomers

Comments

@fviolette
Copy link

Hello,

I tried using astro-tabs within MDX but couldn't get it to work property.
There seems to be an offset with tabs vs panels: https://stackblitz.com/edit/github-tp8cb8?file=src/pages/index.mdx

It works in .astro files though: https://stackblitz.com/edit/github-tp8cb8?file=src%2Fpages%2Fworks.astro,src%2Fpages%2Findex.mdx&on=stackblitz

@JulianCataldo JulianCataldo added bug-fix Something isn't working good first issue Good for newcomers labels Dec 19, 2022
@JulianCataldo
Copy link
Owner

Hello,

Thanks for your input, and the repro!

In fact, MDX is semi-broken, it adds an unnecessary empty panel.
Also, unrelated but there is a "false" class which is not supposed to be here (easy fix).

I keep this open as a reminder, until I get to integrate this component in a new project setup.

@fviolette
Copy link
Author

Duly noted. Thanks ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants