[wp-trac] [WordPress Trac] #40242: Bulk edit tag autocomplete layout error

[WordPress Trac]

#40242: Bulk edit tag autocomplete layout error
----------------------------------------------+----------------------------
 Reporter:  davidbenton                       |       Owner:
     Type:  defect (bug)                      |      Status:  new
 Priority:  normal                            |   Milestone:  4.7.4
Component:  Quick/Bulk Edit                   |     Version:  4.7
 Severity:  normal                            |  Resolution:
 Keywords:  has-screenshots has-patch commit  |     Focuses:  ui,
                                              |  javascript
----------------------------------------------+----------------------------
Changes (by afercia):

 * keywords:  has-screenshots has-patch needs-testing => has-screenshots
     has-patch commit


Comment:

 Tested a bit more and I think the different behaviour between Bulk Edit
 (where the misplacing occurs) and Quick Edit (which works ok) depends on a
 mix of factors.
 Both Quick Edit and Bulk Edit use a hidden form that then gets revealed.
 By the way, the Quick Edit one gets cloned each time with jQuery `clone()`
 so it's a new object each time while the Bulk Edit one gets just moved
 through the DOM. I guess the Quick Edit autocomplete widget is always a
 new fresh instance and thus its options are properly extended, while the
 Bulk Edit autocomplete widget gets its options overridden the second time
 it gets used.
 Regardless, passing always a complete `position` object solves the
 positioning issue. Patch looks good to me. @davidbenton any thoughts? :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40242#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform