Skip to content

[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

Merged
merged 1 commit into from
Jan 3, 2019

Conversation

yceruto
Copy link
Member

@yceruto yceruto commented Dec 25, 2018

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

Copy link
Contributor

@HeahDude HeahDude left a 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

@yceruto
Copy link
Member Author

yceruto commented Dec 25, 2018

Thank you @HeahDude for your quick review and merry Christmas!

Copy link
Contributor

@vudaltsov vudaltsov left a 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
Copy link
Contributor

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)

@yceruto yceruto force-pushed the add_block_prefix_option branch from f0d120f to bd50ac4 Compare January 2, 2019 01:10
@yceruto
Copy link
Member Author

yceruto commented Jan 3, 2019

Doc PR created.

(AppVeyor failure is unrelated)

@fabpot
Copy link
Member

fabpot commented Jan 3, 2019

Thank you @yceruto.

@fabpot fabpot merged commit bd50ac4 into symfony:master Jan 3, 2019
fabpot added a commit that referenced this pull request Jan 3, 2019
…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
@yceruto yceruto deleted the add_block_prefix_option branch January 3, 2019 15:10
javiereguiluz added a commit to symfony/symfony-docs that referenced this pull request Jan 22, 2019
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
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Apr 30, 2019
@fabpot fabpot mentioned this pull request May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants