Skip to content

fix: remove zwsp characters from markdown docs#334

Merged
barjin merged 1 commit into
masterfrom
fix/remove-html-encoded-chars
Oct 30, 2024
Merged

fix: remove zwsp characters from markdown docs#334
barjin merged 1 commit into
masterfrom
fix/remove-html-encoded-chars

Conversation

@barjin
Copy link
Copy Markdown
Member

@barjin barjin commented Oct 30, 2024

Removes unnecessary zero-width space characters from the mdx files. The source of those is unclear - git blame says that they have been there since > 2 years ago. The on-page rendering remains the same with or without those characters.

Closes apify/apify-docs#1262

@barjin barjin self-assigned this Oct 30, 2024
@github-actions github-actions Bot added this to the 101st sprint - Tooling team milestone Oct 30, 2024
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 30, 2024
@B4nan
Copy link
Copy Markdown
Member

B4nan commented Oct 30, 2024

I recall this was needed when the component was the first thing on the line, but maybe things changed in v3.

@B4nan
Copy link
Copy Markdown
Member

B4nan commented Oct 30, 2024

yeah looks like its no longer needed in v3, they removed the whole section of workarounds from the docs

facebook/docusaurus#8352

@barjin
Copy link
Copy Markdown
Member Author

barjin commented Oct 30, 2024

You were once again faster 👍🏽 thanks for the clarification

@barjin barjin merged commit d3e9675 into master Oct 30, 2024
@barjin barjin deleted the fix/remove-html-encoded-chars branch October 30, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Encoded character in description for linked Class link

2 participants