[wp-trac] [WordPress Trac] #36774: Wrong JQuery UI AMD support

[WordPress Trac]

#36774: Wrong JQuery UI AMD support
----------------------------+-----------------------------
 Reporter:  Andrei Kazakou  |      Owner:
     Type:  defect (bug)    |     Status:  new
 Priority:  normal          |  Milestone:  Awaiting Review
Component:  General         |    Version:  4.5.1
 Severity:  normal          |   Keywords:
  Focuses:                  |
----------------------------+-----------------------------
 After integration of Reuire.js into theme, i found some interested
 problem. When system include JQuery UI scripts with Require.js support, i
 catch the error for not existed modules.

 Looks in the /wp-includes/js/jquery/ui/ directory i found some files with
 AMD loading support, but with wrong module names in depending modules.

 For example file '''wp-includes/js/jquery/ui/accordion.min.js''' contain
 this substring
 {{{
 define(["jquery","./core","./widget"],a)
 }}}
 but Require.js can't find core or wodget modules. It happen, because by
 default this modules in wordpress area have names likes "jquery-ui-core"
 and "jquery-ui-widget"

 Please, change all lines in JS files to correct module names, like
 {{{
 define(["jquery","jquery-ui-core","jquery-ui-widget"],a)
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/36774>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform