Skip to content

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

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

feat(Message): Allow further customization of Markdown behavior#636
rebeccaalpert merged 1 commit intopatternfly:mainfrom
rebeccaalpert:markdown-user

Conversation

@rebeccaalpert
Copy link
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
Copy link

patternfly-build commented Aug 4, 2025

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

@thatblindgeye thatblindgeye left a comment

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 on lines +188 to +189
/** Disables markdown parsing for message, allowing only text input */
disableMarkdown?: boolean;
Copy link
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
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
Copy link

github-actions bot commented Aug 4, 2025

🎉 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