Skip to content

feat(Message): Allow further customization of Markdown behavior - #636

Merged
rebeccaalpert merged 1 commit into
patternfly:mainfrom
rebeccaalpert:markdown-user
Aug 4, 2025
Merged

rebeccaalpert merged 1 commit into
patternfly:mainfrom
rebeccaalpert:markdown-user

Conversation

@rebeccaalpert

Copy link
Copy Markdown
Member

Allow turning off markdown parsing (going to text-only) or passing additional props to parser react-markdown, such as disallowedElements. (Using disallowedElements allows for control over what tags can be displayed by parser.) Other options are also now available.

@rebeccaalpert rebeccaalpert added this to the 2025.Q3 milestone Aug 4, 2025
@patternfly-build

patternfly-build commented Aug 4, 2025

Copy link
Copy Markdown

@rebeccaalpert rebeccaalpert removed this from the 2025.Q3 milestone Aug 4, 2025

@thatblindgeye thatblindgeye left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Couple nits below, but not blocking for me - just wanted to mention in case you had any thoughts

Comment thread packages/module/src/Message/Message.tsx Outdated
Comment on lines +188 to +189
/** Disables markdown parsing for message, allowing only text input */
disableMarkdown?: boolean;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Non-blockers:

  • Would we want to include some verbiage in the description for when you may want to disable markdown?
  • react-core conventions speaking: isMarkdownDisabled instead for the prop name?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated prop name! Let's wait on the verbiage for now and update in a little bit if that's ok.

Allow turning off markdown parsing (going to text-only) or passing additional props to parser react-markdown, such as disallowedElements.
@rebeccaalpert
rebeccaalpert merged commit f3176a3 into patternfly:main Aug 4, 2025
7 checks passed
@github-actions

github-actions Bot commented Aug 4, 2025

Copy link
Copy Markdown

🎉 This PR is included in version 6.4.0-prerelease.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

rebeccaalpert added a commit to rebeccaalpert/virtual-assistant that referenced this pull request Oct 24, 2025
…ernfly#636)

Allow turning off markdown parsing (going to text-only) or passing additional props to parser react-markdown, such as disallowedElements.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow for disabling markdown parsing or providing finer-grained control

4 participants