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

fix: add support for markdown syntax in table cells #2037

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

DersJ
Copy link
Contributor

@DersJ DersJ commented Jul 10, 2023

Description

We might want to use markdown syntax in table cells to show links, style text, or do other stuff.

This is handle automatically by showdown, but wasn't captured in our custom TdFlavoredMarkdownTableComponent.

What's included?

  • wrapping table cells in TdFlavoredMarkdownTableComponent with <td-markdown>

This fixes https://teradatacloud.atlassian.net/browse/CCP-2488

Test Steps

  • npm run start
  • check out flavored markdown table
  • table cells contain links and styled text

General Tests for Every PR

  • npm run start still works.
  • npm run lint passes.
  • npm run stylelint passes.
  • npm test passes and code coverage is not lower.
  • npm run build still works.
Screenshots or link to StackBlitz/Plunker
Screenshot 2023-07-10 at 2 46 10 PM

Copy link
Collaborator

@owilliams320 owilliams320 left a comment

Choose a reason for hiding this comment

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

LGMT 🚀

@owilliams320 owilliams320 merged commit a0ff5a6 into main Jul 10, 2023
6 checks passed
@owilliams320 owilliams320 deleted the fix/markdown-in-tables branch July 10, 2023 21:17
@owilliams320
Copy link
Collaborator

🎉 This PR is included in version 6.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants