Skip to content

Rename parameter 'people' to 'items' in example#185

Open
AMDphreak wants to merge 1 commit into
handlebars-lang:masterfrom
AMDphreak:patch-1
Open

Rename parameter 'people' to 'items' in example#185
AMDphreak wants to merge 1 commit into
handlebars-lang:masterfrom
AMDphreak:patch-1

Conversation

@AMDphreak

@AMDphreak AMDphreak commented Jun 8, 2026

Copy link
Copy Markdown

"Block Helpers" example and description don't match

https://handlebarsjs.com/guide/#block-helpers

receives the people

->

receives the items

@jaylinski

Copy link
Copy Markdown
Member

The helper receives the people-array from the input:

So I think the documentation is correct. But the fact that this confused you, may requires some improvement of the wording.

@AMDphreak

AMDphreak commented Jun 17, 2026

Copy link
Copy Markdown
Author

The helper receives the people-array from the input:

So I think the documentation is correct. But the fact that this confused you, may requires some improvement of the wording.

Not to mention the # doesn't exist in the example, "preceding" is misspelled as "preceeding", closing mustache is stated to be a / which is not a mustache at all, and the second parameter is not options but a function(), and likewise fn is mentioned but is nowhere in the code block. What is going on with this example? It's an unusable document.

Screenshot_20260617_131455_Chrome

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