Skip to content

Conversation

@maximilianfalco
Copy link
Contributor

@maximilianfalco maximilianfalco commented Dec 17, 2025

PR App Fix CX-2640

🧰 Changes

Add support for rendering legacy tutorial-tile/recipe magic blocks

Screenshot 2026-01-09 at 13 12 37

🧬 QA & Testing

Create a magic blocks tutorial/recipe block in mdxish. The card should be rendered now.

@maximilianfalco maximilianfalco changed the title Falco/cx 2640 support tutorial tile in mdxish feat(mdxish): support tutorial tile in mdxish Dec 17, 2025
…to falco/cx-2640-support-tutorial-tile-in-mdxish
Copy link
Contributor

@kevinports kevinports left a comment

Choose a reason for hiding this comment

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

The tutorial tile specific code seems good here, but had a question about the table cell parsing logic in this change set.

Comment on lines 97 to 98
/** Parses markdown in table cells */
const cellParser = unified().use(remarkParse).use(remarkGfm);
Copy link
Contributor

Choose a reason for hiding this comment

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

Confused what this change has to do with supporting tutorial tiles in mdxish?
& why would we parse table content with a plain remarkParse/remarkGfm pipeline? I'd expect we'd still want to support readme flavored syntax within a table cell where possible. But I may be missing something here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah oops I think I branched this branch off of a table cell fix @eaglethrost made a while back. Will pull from next to remove this dif

Copy link
Contributor Author

@maximilianfalco maximilianfalco Jan 9, 2026

Choose a reason for hiding this comment

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

Yep okay fixed this by pulling from next -> 234ef04

…to falco/cx-2640-support-tutorial-tile-in-mdxish
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