10up / simple-page-ordering

Order your pages and other hierarchical post types with simple drag and drop right from the standard page list.
https://wordpress.org/plugins/simple-page-ordering/
GNU General Public License v2.0
149 stars 23 forks source link

Ignore input elements in the row #13

Closed helen closed 6 years ago

helen commented 6 years ago

If there are input/form elements in the row, they cannot be interacted with because of drag and drop. There are plenty of other spaces to "grab" the row, so we should be able to safely ignore those elements. This did not affect quick edit in my testing (despite a forum post linked below about it) but worth checking there as well.

.org forum posts:

NateWr commented 6 years ago

This has caused issues for one of my plugins as well. I've tested @oscarssanchez's PR https://github.com/10up/simple-page-ordering/pull/21 and it fixed the issue for me, without causing any noticeable side effects.

If a different approach to solving this is desired, I'd be happy to work up a PR. :+1: