stereobooster / jekyll-press

🚨 [deprecated] Minifier plugin for jekyll. Minifies all html, js, css files. Simple just drop it in solution. No Java required
MIT License
148 stars 13 forks source link

Error in HTML and in console #4

Closed OliverJAsh closed 7 years ago

OliverJAsh commented 12 years ago

In the console I am getting this error from jekyll. This only occurs with jekyll_press:

parse error occurred while processing: /Users/Oliver/Development/Personal/OliverJAsh/site/css/base.css
details: parse error on value ":not(" (error)
copying initial file

I am also getting an error of Uncaught SyntaxError: Unexpected token ILLEGAL in the browser console. Something seems to be going wrong in the minification of the HTML here.

My project where I am witnessing this problems can be found here.

stereobooster commented 12 years ago
parse error occurred while processing: /Users/Oliver/Development/Personal/OliverJAsh/site/css/base.css
details: parse error on value ":not(" (error)
copying initial file

This one is not error, but warning. Which says that there is unsupported css selector :not(. This error occurs in csspool (or in this case csspool-st - my fork of csspool). So instead of minification of base.css it copies original file.

About Uncaught SyntaxError: Unexpected token ILLEGAL i need to investigate.

stereobooster commented 12 years ago

What command do you use to run jekyll?

cd oliverash.me
bundle
bundle exec jekyll source site

It's strange that I got following

You are missing a library required for Markdown. Please run:
  $ [sudo] gem install rdiscount
stereobooster commented 12 years ago

I found error

<script>window.jQuery || document.write('<script src="/components/jquery/jquery.js"><\/script>');</script>

it generates

<script>window.jQuery||document.write('<script src="/components/jquery/jquery.js"></script>')</script>
OliverJAsh commented 12 years ago

Interesting. Why is that happening?

On 6 Oct 2012, at 09:20, stereobooster notifications@github.com wrote:

I found error

it generates

')

— Reply to this email directly or view it on GitHub.

stereobooster commented 12 years ago

Need to use option :inline_script for uglifier. Already fixed it. Try to update html_press gem.

OliverJAsh commented 12 years ago

Thank you!

On 6 Oct 2012, at 10:24, stereobooster notifications@github.com wrote:

Need to use option :inline_script for uglifier. Already fixed it. Try to update html_press gem.

— Reply to this email directly or view it on GitHub.

stereobooster commented 12 years ago

About problem with css parser i reported here tenderlove/csspool#26

OliverJAsh commented 11 years ago

Does csspool appreciate jekyll-press’ exclude property, can you tell me? I'm telling it to ignore my CSS but it's still coming up with the error:

parse error occurred while processing: /Users/Oliver/Development/Personal/OliverJAsh/site/css/base.css
details: parse error on value ":not(" (error)
copying initial file

I actually only need to minify my HTML. Should I be using jekyll-press?

stereobooster commented 11 years ago

I will check it

fbnlsr commented 10 years ago

Regarding this:

details: parse error on value ":not(" (error)

I see that it has already been reported a year ago, and apparently it's still not working. Is there a solution to make this work as it should be?

fbnlsr commented 10 years ago

It's been six months, @stereobooster could you look into it ?

stereobooster commented 7 years ago

project is deprecated https://github.com/stereobooster/jekyll-press/issues/35