Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-17876

print.css should be loaded with eZJSCore (easy to fix)

    Details

      Description

      I'm using webin.

      In extension/ezwebin/design/ezwebin/templates/page_head_style.tpl, you can see that several css files are loaded with ezcss_load. Which is very good (minifying is a very good stuff)

      But at the of this file, you can read:

      <link rel="stylesheet" type="text/css" href={"stylesheets/print.css"|ezdesign} media="print" />
      

      As a result print.css won't be minified, which is a pity...

      A solution would be to delete this line (<link rel="stylesheet" type="text/css" href=

      {"stylesheets/print.css"|ezdesign}

      media="print" />)

      Then adding print.css to the ezcss_load()

      And finally in print.css wrap all the content by:

      @media print {
       
      all-the-content-of-print.css-here
       
      }
      

      A chance to see that for the next ezp release ?

        Activity

        Show
        Jerome Despatis added a comment - https://github.com/ezsystems/ezwebin/pull/8
        Hide
        Carlos Revillo added a comment - - edited

        In reply to comment #024082
        err forget it. that will add print.css to the whole thing and we don't want that. sorry, talked too fast.

        Show
        Carlos Revillo added a comment - - edited In reply to comment #024082 err forget it. that will add print.css to the whole thing and we don't want that. sorry, talked too fast.
        Hide
        Carlos Revillo added a comment -

        and why don't just delete the line you are refering to and add print.css to the frontend css list?

        Show
        Carlos Revillo added a comment - and why don't just delete the line you are refering to and add print.css to the frontend css list?

          People

          • Assignee:
            unknown
            Reporter:
            Jerome Despatis
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: