Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Content-Security-Policy compatibility #288

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

bmihelac
Copy link
Contributor

@bmihelac bmihelac commented May 31, 2024

Summary of Changes:

  • Convert admin/img/icon-alert.svg to a data URI to avoid referencing static files in CSS.
  • Convert rosetta.css to a proper CSS file and move it to the static folder.
  • Pass required variables for JavaScript through the rosetta_settings_js context variable. rosetta_settings_js contains relevant settings and variables for JavaScript.
  • Convert rosetta.js to a proper JS file and move it to the static folder.
  • Move inline style for action-toggle to the stylesheet file.

ref: #287

All Submissions:

  • Are tests passing? (From the root-level of the repository please run pip install tox && tox)
  • I have added or updated a test to cover the changes proposed in this Pull Request

No

  • I have updated the documentation to cover the changes proposed in this Pull Request

No

Convert `admin/img/icon-alert.svg` to data-uri to avoid referencing static files in css
Convert css to proper css file, move it to static folder.
Pass required variables for javascript as `rosetta-settings-js` json.
Convert js to proper js file, move it to static folder.
Move inline style for `action-toggle` to stylesheet file.
@mbi mbi merged commit 2cc05a3 into mbi:develop Jun 2, 2024
4 checks passed
mbi added a commit that referenced this pull request Jun 3, 2024
mbi added a commit that referenced this pull request Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants