Skip to content

Commit

Permalink
Insert ++resource++plone.app.widgets.js resource at the beginning…
Browse files Browse the repository at this point in the history
…, before ``++resource++plone.app.jquery.js`` (but after ``++resource++plone.app.widgets-requirejs-unset.js``).

This makes other installations more predictable when they install resources directly after ``++resource++plone.app.jquery.js``.
  • Loading branch information
thet committed Jun 28, 2017
1 parent 4d4baca commit b0c4b85
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 5 additions & 3 deletions CHANGES.rst
Expand Up @@ -4,9 +4,11 @@ Changelog
1.10 (unreleased)
-----------------

- failsafe Unset and reset ``define`` and ``require``
[petschki]
- Insert ``++resource++plone.app.widgets.js`` resource at the beginning, before ``++resource++plone.app.jquery.js`` (but after ``++resource++plone.app.widgets-requirejs-unset.js``).
This makes other installations more predictable when they install resources directly after ``++resource++plone.app.jquery.js``.
[thet]


- PEP 8.
[thet]

Expand All @@ -17,7 +19,7 @@ Changelog
- Unset and reset ``define`` and ``require`` before and after the plone.app.widgets JavaScript bundle.
This avoids the infamous ``Mismatched anonymous define`` error with unbundled JavaScript supporting RequireJS.
See: http://requirejs.org/docs/errors.html#mismatch
[thet]
[thet, petschki]

- Remove Plone 5 specific zcml conditions
[petschki]
Expand Down
2 changes: 1 addition & 1 deletion plone/app/widgets/profiles/default/jsregistry.xml
Expand Up @@ -17,7 +17,7 @@

<javascript
id="++resource++plone.app.widgets.js"
insert-after="++resource++plone.app.jquery.js"
insert-before="*"
cacheable="True"
compression="none"
cookable="True"
Expand Down

0 comments on commit b0c4b85

Please sign in to comment.