Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minimal description of tag {form}{/form} #998

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Minimal description of tag {form}{/form} #998

merged 1 commit into from
Apr 7, 2023

Conversation

mildabre
Copy link
Contributor

@mildabre mildabre commented Apr 5, 2023

Latte tag {form}{/form} is insufficiently described in the documentation. The tag is mentioned in Latte https://latte.nette.org/cs/tags, in the paragraph "Only available with Nette Forms". Here is just a link that leads to the Nette Forms documentation: https://doc.nette.org/cs/forms/rendering#toc-latte, but again we can't find any definition of this tag.

Since the tag {form "formName"}{/form} is essentially an alternative notation for <form n:name="formName" ...>, which is well described in the documentation, I suggest inserting only a small separate paragraph where it would be clearly explained .

  • change correspondingly the link from documentation Latte

Latte tag {form}{/form} is insufficiently described in the documentation. The tag is mentioned in Latte https://latte.nette.org/cs/tags, in the paragraph "Only available with Nette Forms". Here is just a link that leads to the Nette Forms documentation: https://doc.nette.org/cs/forms/rendering#toc-latte, but again we can't find any definition of this tag.

Since the tag {form "formName"}{/form} is essentially an alternative notation for <form n:name="formName" ...>, which is well described in the documentation, I suggest inserting only a small separate paragraph where it would be clearly explained .
@dg dg merged commit bd4681e into nette:master Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 18, 2023
dg pushed a commit that referenced this pull request Apr 18, 2023
dg pushed a commit that referenced this pull request Apr 18, 2023
dg pushed a commit that referenced this pull request Apr 25, 2023
dg pushed a commit that referenced this pull request Apr 27, 2023
dg pushed a commit that referenced this pull request May 22, 2023
dg pushed a commit that referenced this pull request May 29, 2023
dg pushed a commit that referenced this pull request Oct 3, 2023
dg pushed a commit that referenced this pull request Oct 24, 2023
dg pushed a commit that referenced this pull request Oct 24, 2023
dg pushed a commit that referenced this pull request Oct 24, 2023
dg pushed a commit that referenced this pull request Apr 18, 2024
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