-
Notifications
You must be signed in to change notification settings - Fork 135
Closed
Copy link
Description
From https://github.com/Automattic/Edit-Flow/pull/649/files#r645296481
Expected/Desired Behavior
Remove the redundancy code.
Code:
Details
tl;dr: We can remove this jQuery listener because (1) jQuery selector is no longer found, and (2) there is no issue with custom statuses and Quick Edit right now.
As mentioned in the comment here, this click event is triggered if Quick Edit links are clicked in http://valet-vipgo-3279.test/wp-admin/edit.php
However, Quick Edit is currently a button element. Its HTML looks like this:
<button type="button" class="button-link editinline" aria-label="Quick edit “Add a new post – set this for future” inline" aria-expanded="false">Quick Edit</button>Also, in the browser console, I can double check that with these functions:
jQuery('#the-list a.editinline').lengthreturns 0jQuery('#the-list button.editinline').lengthreturns20(or the number of posts on this screen).
That said, I am not seeing any issue with Quick Edit and custom statuses here.