Debug assets ignores media type for stylesheets #2625

Closed
moser opened this Issue Aug 22, 2011 · 1 comment

Comments

Projects
None yet
3 participants
@moser

moser commented Aug 22, 2011

When asset debugging is activated, every stylesheet link tag has media="screen".
The following code in my layout

= stylesheet_link_tag 'application', :media => "all"
= stylesheet_link_tag 'print/print', :media => "print"

produces

<link href="/assets/app.css?body=1" media="screen" rel="stylesheet" type="text/css" /> 
<link href="/assets/jquery-ui-1.8.5.custom.css?body=1" media="screen" rel="stylesheet" type="text/css" /> 
<link href="/assets/application.css?body=1" media="screen" rel="stylesheet" type="text/css" /> 
<link href="/assets/print/print.css?body=1" media="screen" rel="stylesheet" type="text/css" /> 

when the debug_assets=1 parameter is passed.
Without the parameter it works as expected:

<link href="/assets/application.css" media="all" rel="stylesheet" type="text/css" /> 
<link href="/assets/print/print.css" media="print" rel="stylesheet" type="text/css" /> 

guilleiguaran added a commit to guilleiguaran/rails that referenced this issue Aug 24, 2011

@guilleiguaran

This comment has been minimized.

Show comment Hide comment
@guilleiguaran

guilleiguaran Aug 24, 2011

Owner

Check #2625

Owner

guilleiguaran commented Aug 24, 2011

Check #2625

arunagw pushed a commit to arunagw/rails that referenced this issue Aug 25, 2011

Merge pull request #2668 from guilleiguaran/debug-assets-media-type
Debug assets shouldn't ignore media type for stylesheets. Closes #2625

arunagw pushed a commit to arunagw/rails that referenced this issue Aug 25, 2011

Merge pull request #2668 from guilleiguaran/debug-assets-media-type
Debug assets shouldn't ignore media type for stylesheets. Closes #2625
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment