Skip to content
This repository has been archived by the owner on May 11, 2021. It is now read-only.

Remove inline NEXUS definition #31

Closed
wants to merge 1 commit into from

Conversation

graingert
Copy link

To comply with the Django 1.10 JavaScript deployment guidelines and CSP without unsafe-inline.

@@ -81,19 +81,22 @@
* Public, $.facebox methods
*/

var nexusMediaPreix = $('#nexus-facebox-constants').attr('data-nexus-media-prefix'),
closeImage = nexusMediaPrefix + 'nexus/img/facebox/closelabel.gif';

$.extend($.facebox, {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eventually this should be moved into nexus's own JS so you can update facebox.js to 1.3 from https://github.com/defunkt/facebox

Sadly this version of 1.2 and the upstream version have significant changes

@adamchainz
Copy link
Owner

I added the HTML escaping and fixed your typo nexusMediaPreix, as well as added a HISTORY note. Merged in 3ca54d2 , thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants