s/require/define/ in a few more files
closes: CNVS-36299 This change makes it so they Don’t load their deps in a separate chunk so there is not 2 round trips to get each. RequireJS would bundle them into one file but webpack treats an amd require([]... as a split point, different from an amd define([]... Test plan: * Run webpack * you should see less unnamed, numbered chunks Change-Id: I512fe2759b18f729532ecee2c9c233fea4319dd3 Reviewed-on: https://gerrit.instructure.com/108448 Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Ryan Shaw <ryan@instructure.com> QA-Review: Ryan Shaw <ryan@instructure.com>
This commit is contained in:
parent
ed1e14e5ac
commit
e68a26c507
|
@ -1,6 +1,6 @@
|
|||
# listens to clicks on links that would send you to a url that is handled by a loaded backbone router
|
||||
# and bypasses doing a real load to that new page.
|
||||
require [
|
||||
define [
|
||||
'jquery'
|
||||
'underscore'
|
||||
'Backbone'
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
require [
|
||||
define [
|
||||
'jquery'
|
||||
], ($) ->
|
||||
# Makes toggle components behave like buttons for a11y
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
require([
|
||||
define([
|
||||
'i18n!accounts' /* I18n.t */,
|
||||
'jquery' /* $ */,
|
||||
'str/htmlEscape',
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
require([
|
||||
define([
|
||||
'i18n!accounts' /* I18n.t */,
|
||||
'jquery' /* $ */,
|
||||
'compiled/util/addPrivacyLinkToDialog',
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
require([ 'jquery', 'canvas_quizzes/apps/statistics' ], function($, app) {
|
||||
define([ 'jquery', 'canvas_quizzes/apps/statistics' ], function($, app) {
|
||||
app.configure({
|
||||
ajax: $.ajax,
|
||||
loadOnStartup: true,
|
||||
|
|
Loading…
Reference in New Issue