[buddypress-trac] [BuddyPress] #5209: enqueue bp_core_confirmation_js

buddypress-trac noreply at wordpress.org
Tue Oct 22 13:58:06 UTC 2013

#5209: enqueue bp_core_confirmation_js
 Reporter:  bassjobsen    |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Core          |     Version:  1.8.1
 Severity:  normal        |  Resolution:
 Keywords:                |

Comment (by boonebgorges):

 I'm not totally sure why we do it this way (rather than proper enqueuing).
 I guess the idea is that there are places where we would want the .confirm
 JS but we don't want to load our entire script library. That makes sense,
 but in that case why not just have a separate .js file for confirm links?
 It'll get cached pretty heavily, so I can't imagine a huge performance

 Would like an opinion from another dev before moving forward with it.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5209#comment:1>
BuddyPress <http://buddypress.org/>

More information about the buddypress-trac mailing list