-
Notifications
You must be signed in to change notification settings - Fork 135
Description
From https://github.com/Automattic/Edit-Flow/pull/649/files
Description
Currently, this selector is used in 3 jQuery files but none of them seems relevant:
$('#posts-filter input[type="submit"]').on( 'mousedown', function(e){
t.revert();
});We will need to review and clean up this.
Details
File 1. custom-status-configure.js
From https://github.com/Automattic/Edit-Flow/pull/649/files#r644641007
On this page http://valet-vipgo-3279.test/wp-admin/admin.php?page=ef-editorial-metadata-settings, there is no exact
$('#posts-filter input[type="submit"]')element though there is an element with#posts-filter.
This seems redundant and I can not find how this event is triggered.
Agree - probably leftover code.
(From @mikeyarce)
File 2. user-groups-configure.js
From https://github.com/Automattic/Edit-Flow/pull/649/files#r644571646
This seems redundant as there is no element with this ID
posts-filterin this page http://valet-vipgo-3279.test/wp-admin/admin.php?page=ef-user-groups-settingsThen this event is not triggered.
Yeah this seems redundant. It looks like other pages like wp-admin/admin.php?page=ef-editorial-metadata-settings use this but not User Groups
(From @mikeyarce)
File 3. editorial-metadata-configure.js
From https://github.com/Automattic/Edit-Flow/pull/649/files#r644641007
On this page http://valet-vipgo-3279.test/wp-admin/admin.php?page=ef-editorial-metadata-settings, there is no exact
$('#posts-filter input[type="submit"]')element though there is an element with#posts-filter.This seems redundant and I can not find how this event is triggered.
The #posts-filter element is defined here: