remove inline style and script

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
5 years ago
3 years ago

People

(Reporter: pomax, Assigned: zak.hassan1010)

Tracking

Details

(Reporter)

Description

5 years ago
we have several places where we have inline script and inline styles (i.e. not loaded from .js or .css files) in our pages, and we should really try to refactor this so that this isn't the case anywhere anymore.

This ticket can act as a meta ticket while we do an inventory of locations that need fixing, blocked on individual refactors.
(Assignee)

Comment 1

5 years ago
If no one is working on this then can you assign this to me.
Sure.  Please start by doing an audit to find all instances of this, and report back in the bug for follow-up with Pomax on how to fix.  Thanks.
Assignee: nobody → zak.hassan1010
Status: NEW → ASSIGNED
(Assignee)

Comment 3

5 years ago
> The following files use inline css: using this command 'git grep 'style="'|awk '{print $1}'|uniq'

learning_projects/o2_thepitch.html
learning_projects/portfoliomaker.html
public/friendlycode/test/test-live-preview.js
public/friendlycode/vendor/codemirror3/mode/htmlmixed/index.html
public/friendlycode/vendor/codemirror3/mode/xml/index.html
public/friendlycode/vendor/slowparse/spec/index.html
public/friendlycode/vendor/slowparse/vendor/codemirror2/lib/codemirror.js
public/learning_projects/bunnies/source.svg
views/friendlycode/templates/confirm-dialog.html
views/friendlycode/templates/error-dialog.html
views/friendlycode/templates/publish-dialog.html
views/layout.html


For inline javascript: Used this command "git grep 'javascript:'|awk '{print $1}'" 

public/friendlycode/vendor/slowparse/test/test-tree-inspectors.js
public/learning_projects/s2r/style.css

Note: public/learning_projects/s2r/style.css: searches for inline javascript and sets some css on it.
I'll close this and we'll fix when we deal with CSP warnings in Bramble.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.