Skip to content

Adding Upgrading Section #15979

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

Closed
wants to merge 1 commit into from
Closed

Conversation

ThomasLandauer
Copy link
Contributor

This text is (somewhat) a duplicate of https://symfony.com/doc/current/form/bootstrap5.html#error-messages - if you agree in principle that adding an upgrade section is a good idea, I'll see if I can improve this.

Second list item would be to remove all radio-custom, checkbox-custom, and switch-custom https://symfony.com/doc/current/form/bootstrap4.html#custom-forms

Plus maybe a link to https://getbootstrap.com/docs/5.1/migration/

This text is (somewhat) a duplicate of https://symfony.com/doc/current/form/bootstrap5.html#error-messages - if you agree *in principle* that adding an upgrade section is a good idea, I'll see if I can improve this.

Second list item would be to remove all `radio-custom`, `checkbox-custom`, and `switch-custom` https://symfony.com/doc/current/form/bootstrap4.html#custom-forms

Plus maybe a link to https://getbootstrap.com/docs/5.1/migration/
@ThomasLandauer
Copy link
Contributor Author

Linting error: The code block is meant to be indented (for the list) - what is the right syntax?

@ThomasLandauer
Copy link
Contributor Author

Actually, the text isn't quite right. Since form_errors is not included in form_label anymore, you need to manually add form_errors!

@javiereguiluz
Copy link
Member

I'm closing this old issue as "won't fix".

This is not wrong ... but would require a certain amount of maintenance work. So, let's do this only if we receive enough feedback from the community about this. So far, I think we didn't receive any comments here or in Symfony Slack chat. So, let's keep waiting for feedback.

Thanks for understanding.

@ThomasLandauer ThomasLandauer deleted the patch-26 branch January 18, 2024 14:01
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.

3 participants