Skip to content

Adds support for url embeds for message and tab tags#146

Merged
MShultz merged 6 commits intomainfrom
fix/check-for-parent-tags
Jun 1, 2025
Merged

Adds support for url embeds for message and tab tags#146
MShultz merged 6 commits intomainfrom
fix/check-for-parent-tags

Conversation

@MShultz
Copy link
Contributor

@MShultz MShultz commented May 26, 2025

Updates text message, message, tabs, and tab tags to correctly render url embeds of the same keyword.
There are some special cases where rendering fails, such as attempting to use a child keyword as the text for an url embedd in an otherwise valid parent/child relationship. Such a case is not supported with the BBCode engine at this time. Examples:
[textmessage][message=me]This message won't render the bbcode [message](https://google.com)[/message][/textmessage]
[tabs][tab]This tab also won't render the bbcode [tab](https://google.com)[/tab][/tabs]

Also adds optional chaining to all the tags to avoid undefined errors.

@MShultz MShultz force-pushed the fix/check-for-parent-tags branch from 996d319 to 1fc3180 Compare May 29, 2025 02:12
@MShultz MShultz marked this pull request as ready for review May 29, 2025 02:33
@MShultz MShultz changed the title Adds conditional parsing for [message] tag. Adds support for url embeds for message and tab tags May 29, 2025
@MShultz MShultz merged commit 12fc2a3 into main Jun 1, 2025
3 checks passed
@MShultz MShultz deleted the fix/check-for-parent-tags branch June 1, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants