The current documentation only indicates that relative paths to css files are supported, however it appears that stream wrapper URIs work fine as well (e.g. public://css_injector/css_injector_1.css).
Support was added here: #927406: Colorable themes fail when CSS aggregation is enabled due to improper stream wrapper usage
A css_injector issue that uses a stream wrapper URI: #1044670: Fix file handling resulting in Incorrect path to css file on Windows
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | drupal-1222794-8.patch | 2.05 KB | jhodgdon |
| #8 | drupal-1222794-8-D8.patch | 2.05 KB | gapple |
| #6 | drupal-1222794-D8.patch | 2.05 KB | gapple |
| #3 | drupal-1222794-3.patch | 2.04 KB | gapple |
| #1 | drupal-1222794-1.patch | 1.61 KB | gapple |
Comments
Comment #1
gappleComment #3
gappleHere is a proper Git patch.
The only change is the first sentence of the description of the $data parameter when $options['type'] is 'file':
Changes to
Comment #4
gappleforgot status change
Comment #5
jhodgdonI know this was in the original text wrong, but there's a slight punctuation problem:
wrapper URI, e.g.,
should be
wrapper URI; e.g.,
Actually, I think it should say:
The path to the CSS file relative to the base_path(), or a stream wrapper URI. For example: "modules/devel/devel.css" or "public://generated_css/stylesheet_1.css".
Also, we patch vs. 8.x and backport to 7.x. Thanks!
Comment #6
gappleThat's definitely a bit easier to read.
Here's an updated patch for D8.
I was able to cherry-pick the commit without issue, so I'm sure the patch will still apply to D7 cleanly as well.
Comment #7
gapplestatus
Comment #8
gappleJust noticed some end-of-line whitespace snuck in to the previous patch
Comment #9
jhodgdonTo get a patch tested, it can't have a -D# on the patch name (which is noted in the attachment field help text below where you attach files). So, re-uploading the previous patch with a different name.
And the patch looks fine to me for d7/8, as long as the test bot agrees.
Comment #10
webchickCommitted and pushed to 8.x and 7.x. Thanks!