[wp-trac] [WordPress Trac] #47510: Developer tool - Best Practice - input comment field Does not use passive listeners

WordPress Trac noreply at wordpress.org
Sun Jan 3 23:54:14 UTC 2021


#47510: Developer tool - Best Practice - input comment field Does not use passive
listeners
------------------------------------+--------------------------------------
 Reporter:  efilippucci             |       Owner:  (none)
     Type:  defect (bug)            |      Status:  closed
 Priority:  normal                  |   Milestone:
Component:  Comments                |     Version:  5.2.1
 Severity:  normal                  |  Resolution:  wontfix
 Keywords:  has-patch needs-        |     Focuses:  javascript, performance
  refresh                           |
------------------------------------+--------------------------------------
Changes (by peterwilsoncc):

 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Future Release =>


Comment:

 I've decided to close this off with the `wontfix` status as the code is
 incompatible with passive listeners.

 As mentioned above, the event listener uses `event.preventDefault()` to
 stop browsers from following the link. Using a passive listener would
 result in browsers following the link.

 Discussion can continue on closed tickets but unless browser behavior
 changes to allow both passive listeners and preventing the default action,
 the dev-tools report remains a false positive.

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


More information about the wp-trac mailing list