[wp-trac] [WordPress Trac] #42321: Menu title field does not clear invalid class
WordPress Trac
noreply at wordpress.org
Fri Jan 18 09:10:40 UTC 2019
#42321: Menu title field does not clear invalid class
---------------------------------------------+-----------------------------
Reporter: ritukaushal5693 | Owner: welcher
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 5.1
Component: Menus | Version: 4.8.2
Severity: normal | Resolution:
Keywords: good-first-bug has-patch commit | Focuses: ui,
| administration
---------------------------------------------+-----------------------------
Changes (by afercia):
* keywords: good-first-bug has-patch needs-refresh => good-first-bug has-
patch commit
Comment:
[attachment:"42321.diff"] addresses the points above.
Also, I'd propose to keep using the `form-invalid` CSS class, as the new
one on the single input doesn't account for the focus style: when invalid
and focused, two box-shadows (red and blue) were applied to the field. We
shouldn't change this part as it would require some unnecessary CSS
refactoring. As far as I can tell, there are no concerns for unintended
side effects as `form-invalid` is only applied to the menu name field
parent.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/42321#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list