-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Form] Add new block_prefix option for an easy form theming #29680
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
Conversation
yceruto
commented
Dec 25, 2018
•
edited
Loading
edited
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | #29651 |
License | MIT |
Doc PR | symfony/symfony-docs#10835 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks and merry Christmas <3
Thank you @HeahDude for your quick review and merry Christmas! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea, love it!
multiple data | ||
placeholder disabled | ||
preferred_choices help | ||
help_attr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmpf this table rst layout makes it hard for small and readable diffs, because in this case only one string is changed (added)
f0d120f
to
bd50ac4
Compare
Doc PR created. (AppVeyor failure is unrelated) |
Thank you @yceruto. |
…eming (yceruto) This PR was merged into the 4.3-dev branch. Discussion ---------- [Form] Add new block_prefix option for an easy form theming | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29651 | License | MIT | Doc PR | symfony/symfony-docs#10835 Commits ------- bd50ac4 Add block_prefix option for an easy form theming
This PR was squashed before being merged into the master branch (closes #10835). Discussion ---------- [Form] Documenting the block_prefix option Documenting new feature: symfony/symfony#29680 Commits ------- d9382d4 [Form] Documenting the block_prefix option