[wp-trac] [WordPress Trac] #39516: Media grid upload errors display below the grid.
WordPress Trac
noreply at wordpress.org
Fri Jan 27 19:38:32 UTC 2017
#39516: Media grid upload errors display below the grid.
------------------------------+--------------------------------------------
Reporter: joemcgill | Owner: afercia
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 4.7.3
Component: Media | Version: 4.6
Severity: normal | Resolution:
Keywords: has-patch needs- | Focuses: ui, accessibility, javascript
testing |
------------------------------+--------------------------------------------
Comment (by MatheusGimenez):
Replying to [comment:12 adamsilverstein]:
> @MatheusGimenez Thanks for the patch, it looks good!
>
> A couple of code comments: for the model id check you should use a
[https://make.wordpress.org/core/handbook/best-practices/coding-
standards/javascript/#equality strict equality check] and also switch the
order to use [https://make.wordpress.org/core/handbook/best-practices
/coding-standards/javascript/#yoda-conditions Yoda conditions].
Hi, i applied the code standards in the last patch.
Thanks for the feedback! :)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/39516#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list