Skip to content

Add back in the 'Pulse' bbcode example script#12146

Open
seanhagen wants to merge 2 commits into
godotengine:masterfrom
seanhagen:fix/bring-back-pulse-bbcode-example
Open

Add back in the 'Pulse' bbcode example script#12146
seanhagen wants to merge 2 commits into
godotengine:masterfrom
seanhagen:fix/bring-back-pulse-bbcode-example

Conversation

@seanhagen

Copy link
Copy Markdown

Seems to have disappeared at some point between now and the v3.2 docs, though it's still referenced in the example below the GDScripts. This just adds back in the script ( and changes tool to @tool ).

Seems to have disappeared at some point between now and the v3.2 docs,
though it's still referenced in the example below the GDScripts. This
just adds back in the script ( and changes `tool` to `@tool` ).
@seanhagen
seanhagen requested review from a team as code owners July 14, 2026 17:37
@Calinou

Calinou commented Jul 15, 2026

Copy link
Copy Markdown
Member

That specific example was intentionally removed as it's superseded by the built-in [pulse] tag added in 4.2:

I don't think we should readd it, especially since we already have other examples listed below. I would only remove the [pulse] usage from the example usage block at the end of the page instead.

@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:gui labels Jul 15, 2026
@seanhagen

Copy link
Copy Markdown
Author

Alrighty, I can re-write the documentation so that's a bit more clear then.

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me.

@AThousandShips

Copy link
Copy Markdown
Member

Title needs an update

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

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation enhancement topic:gui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants