- Issue created by @gbotis
- Merge request !615Issue #3515257 Remove special handling for description for option based elements β (Merged) created by Unnamed author
- Merge request !616Issue #3515257 Remove code moving descpriton to prefix/suffix if display is before/invisible β (Open) created by Unnamed author
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
A markup change like this could make layout changes in cases of customization. So, I doubt this change will be made in 6.2.x.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Please fix the coding standards issues. Why is the test change needed?
- π¬π·Greece gbotis
@liam-morland
You're absolutely right that a markup change like this could affect customized layouts, so I agree it likely shouldn't go into 6.2.x. Still, it's helpful to have the patch available for anyone who wants the corrected behavior prior to updating to 6.3.The test change was probably an artifact of resolving merge conflicts β Iβll double-check and clean that up along with the coding standards. Thanks for the catch!
- π¬π·Greece gbotis
Should be ok now, 615 targeting 6.3.x and 616 targeting 6.2.9
- πΊπΈUnited States jrockowitz Brooklyn, NY
Here is the change record. With tests passing, I am fine with merging this.
-
jrockowitz β
committed 9ac9a166 on 6.3.x authored by
gbotis β
Issue #3515257: Checkboxes/Radios with description display set to "...
-
jrockowitz β
committed 9ac9a166 on 6.3.x authored by
gbotis β
- πΊπΈUnited States jrockowitz Brooklyn, NY
Since we are changing the $form array, this should be only committed to 6.3.x