Skip to content

Conversation

@kanemu
Copy link
Contributor

@kanemu kanemu commented Aug 26, 2023

Fix data.hProperties not working for tableCell

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

This pull request addresses an issue where the data.hProperties property was not being properly applied to tableCell elements in the mdast-util-to-hast library. The problem originates from the loop processing of children within the table-row.js file, where the state.applyData function is being passed the parent tableRow instead of the tableCell itself.

@github-actions github-actions bot added the 👋 phase/new Post is being triaged automatically label Aug 26, 2023
@github-actions

This comment has been minimized.

@github-actions github-actions bot added 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Aug 26, 2023
Signed-off-by: Titus <tituswormer@gmail.com>
@wooorm wooorm changed the title Fix data.hProperties not working for tableCell Fix hProperties on tableCell Aug 28, 2023
@wooorm wooorm merged commit b186995 into syntax-tree:main Aug 28, 2023
@wooorm wooorm added 🐛 type/bug This is a problem 💪 phase/solved Post is done labels Aug 28, 2023
@github-actions

This comment has been minimized.

@github-actions github-actions bot removed the 🤞 phase/open Post is being triaged manually label Aug 28, 2023
@wooorm
Copy link
Member

wooorm commented Aug 28, 2023

Thank you!! https://github.com/syntax-tree/mdast-util-to-hast/releases/tag/13.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💪 phase/solved Post is done 🐛 type/bug This is a problem

Development

Successfully merging this pull request may close these issues.

2 participants