Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Stylesheets not handled correctly

#14

The stylesheets helper outputs print and screen stylesheets in wrong order without appropriate media statement. Specifying media and sheet order doesn't work because .scss aren't handled when specifying sheets.

lib/staticmatic/helpers.rb

Reported by Mark Robinson · September 14th, 2010 @ 05:11 PM

State: new
Milestone: none
Assigned to: Stephen Bartholomew Stephen Bartholomew

Activity

  1. Mark Robinson
    Mark Robinson

    I just hacked it to work by adding this line around line 61:

    File.exist?(File.join(@staticmatic.src_dir, 'stylesheets', "#{file}.scss"))

    September 14th, 2010 @ 08:14 PM

  2. Stephen Bartholomew
    Stephen Bartholomew

    This should be working in the pre-release. Can you try and let me know?

    If it's still not working, can you post code samples?

    Cheers!

    September 22nd, 2010 @ 08:18 PM

  3. Mark Robinson
    Mark Robinson

    I just tried with the latest pre-release 0.11.0 and still not working.

    in layout I have
    = stylesheets :ie

    that produces:

    <link href="/stylesheets/screen.css" media="all" rel="stylesheet" type="text/css"/><link href="/stylesheets/ie.css" media="all" rel="stylesheet" type="text/css"/><link href="/stylesheets/print.css" media="all" rel="stylesheet" type="text/css"/>
    

    What I would like is:

    <link href="/stylesheets/print.css" media="print" rel="stylesheet" type="text/css"/>
    <link href="/stylesheets/screen.css" media="screen" rel="stylesheet" type="text/css"/>
    <!--[if lt IE 8]>
      <link href="/stylesheets/ie.css" media="all" rel="stylesheet" type="text/css"/>
    <![endif]-->
    

    September 29th, 2010 @ 09:07 PM

  4. Mark Robinson
    Mark Robinson

    Sorry, just reread that. There shouldn't be :ie after =stylesheets

    And yes, you've fixed the bit I suggested in my first comment but not the problem above if you want to take a look at it.

    Thanks

    September 29th, 2010 @ 09:11 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.