Update validation.rst#4789
Conversation
| Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | 2.3 | Fixed tickets | Some fixes can be a bit "controversial". Due to PR https://github.com/symfony/symfony-docs/pull/4779/files not all changes were made. Something to keep in mind is what due to https://github.com/symfony/symfony-docs/pull/4779/files using `Assert\...` if arguments are kept on the same line the possibility of horizontal scrollbar (especially on the smaller screens) is quite high. Also for most use cases I think adding PHP constraints is better if arguments are on different lines ("easier to read" and easier to spot errors
|
I would like to split line 344 to remove horizontal scrollbar, but wasn't sure of the best way to do it |
book/validation.rst
Outdated
There was a problem hiding this comment.
While it's not your fault, you should remove the serial comma before "or"
There was a problem hiding this comment.
Yea, I thought it wasn't needed but just left it as is, will fix on the next commit.
|
|
Yes, thank you, completely slipped my mind that is how you do in YAML |
book/validation.rst
Outdated
There was a problem hiding this comment.
You need to add the former label to be BC:
.. _translation-constraint-messages:
Translating Constraint Messages
-------------------------------There was a problem hiding this comment.
Thanks, will change it now
|
What do we do here? |
|
Closing as none of the code blocks are violating the 85 character limit we've decided on after this PR was opened (only the schema definitions do, but we don't care about them). The only other change in here is implementing UserInterface, which is already done. |
Some fixes can be a bit "controversial".
Due to PR https://github.com/symfony/symfony-docs/pull/4779/files not all changes were made.
Something to keep in mind is what due to https://github.com/symfony/symfony-docs/pull/4779/files using
Assert\...if arguments are kept on the same line the possibility of horizontal scrollbar (especially on the smaller screens) is quite high.Also for most use cases I think adding PHP constraints is better if arguments are on different lines ("easier to read" and easier to spot errors