Improve the semantic structure of admin widgets.
Some of the widgets currently in the admin use <p> tags as their wrapper elements.
The current structure uses <p> tags, and for fields that can span multiple lines, such as DateTime and URL fields, <br> tags are used to provide visual line breaks.
Since <p> tags represent paragraphs, they are not suitable as wrappers for elements containing inputs, as is the case here. I believe this tag should be changed to a <div>.
Additionally, <br> tags are used for visual line breaks, but it would be more appropriate to remove them and implement the spacing using CSS.
Change History
(11)
| Triage Stage: |
Unreviewed → Accepted
|
| Owner: |
set to Antoliny
|
| Status: |
new → assigned
|
| Needs tests: |
set
|
| Patch needs improvement: |
set
|
| Needs tests: |
unset
|
| Patch needs improvement: |
unset
|
| Patch needs improvement: |
set
|
| Patch needs improvement: |
unset
|
| Patch needs improvement: |
set
|
| Patch needs improvement: |
unset
|
| Patch needs improvement: |
set
|
| Patch needs improvement: |
unset
|
PR