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

Add docs regarding .progress, inline style attrs, and CSP #18080

Merged
merged 1 commit into from
Oct 30, 2015

Conversation

cvrebert
Copy link
Collaborator

Closes #17785.
CC: @mdo @patrickhlauke

@@ -8,6 +8,11 @@ <h1 id="progress" class="page-header">Progress bars</h1>
<p>Progress bars use CSS3 transitions and animations to achieve some of their effects. These features are not supported in Internet Explorer 9 and below or older versions of Firefox. Opera 12 does not support animations.</p>
</div>

<div class="bs-callout bs-callout-info">
<h4 id="callout-progress-csp">Content Security Policy (CSP) compatibility</h4>
<p>If your website has a <a href="https://developer.mozilla.org/en-US/docs/Web/Security/CSP">Content Security Policy (CSP)</a> which doesn't allow <code>style-src 'unsafe-inline'</code>, then you won't be able to use inline <code>style</code> attributes to set progress bar widths as shown in our examples below. Alternative methods for setting the widths that are compatible with strict CSPs include using a little custom JavaScript or using custom CSS classes.</p>
Copy link
Member

Choose a reason for hiding this comment

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

maybe using JavaScript (to inject inline styles) or CSS classes ?

@cvrebert
Copy link
Collaborator Author

@patrickhlauke Added a little bit to the JS phrasing.

@patrickhlauke
Copy link
Member

LGTM

cvrebert added a commit that referenced this pull request Oct 30, 2015
Add docs regarding .progress, inline style attrs, and CSP
@cvrebert cvrebert merged commit a367408 into master Oct 30, 2015
@cvrebert cvrebert deleted the csp-progress branch October 30, 2015 08:45
@mdo mdo mentioned this pull request Oct 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants