Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

generate css with travis #229

Merged
merged 12 commits into from Feb 15, 2019
Merged

generate css with travis #229

merged 12 commits into from Feb 15, 2019

Conversation

DO1JLR
Copy link
Collaborator

@DO1JLR DO1JLR commented Feb 14, 2019

Eventually this could work. We will see...

The script from @ottojo probably won't generate css.

Eventually this could work. We will see...
@DO1JLR DO1JLR added the WIP Work In Progress label Feb 14, 2019
@DO1JLR
Copy link
Collaborator Author

DO1JLR commented Feb 14, 2019

Issue #220

@DO1JLR DO1JLR removed the WIP Work In Progress label Feb 14, 2019
@toolbox-bot
Copy link

This don't need a extra script for minify css
@toolbox-bot
Copy link

@DO1JLR DO1JLR added the WIP Work In Progress label Feb 15, 2019
@toolbox-bot
Copy link

Now lektor detects if the file is local available. If it is, it will use it.
If not, it will use the remote file and linking to the local font folders, because not every browser will accept the remote fonts that easy on localhost.
@toolbox-bot
Copy link

@DO1JLR DO1JLR requested a review from a team February 15, 2019 10:22
@DO1JLR DO1JLR added Please Review Please review this pull request and removed WIP Work In Progress labels Feb 15, 2019
@maxbachmann
Copy link
Contributor

maxbachmann commented Feb 15, 2019

I already searched the function to check if a file exists locally but could not find it glad you found one :)

maxbachmann
maxbachmann previously approved these changes Feb 15, 2019
Copy link
Contributor

@maxbachmann maxbachmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love it :)

@maxbachmann maxbachmann dismissed their stale review February 15, 2019 16:34

just tested the new description and lektor clear does not work for me

@toolbox-bot
Copy link

@maxbachmann
Copy link
Contributor

ah already though the lektor clear is wrong :)

lektor does not trigger a new build only if a file like main.min.css
suddenly appear or dissappear. It need a manual trigger like deleting
the cache or modify the templeates.

make will now delete the cache.
@DO1JLR
Copy link
Collaborator Author

DO1JLR commented Feb 15, 2019

ah already though the lektor clear is wrong :)

Should be fixed with 9cf1b86, we will see what travis will say!

@toolbox-bot
Copy link

@DO1JLR
Copy link
Collaborator Author

DO1JLR commented Feb 15, 2019

I already searched the function to check if a file exists locally but could not find it glad you found one :)

I am happy about this too. |asseturl only work in the asset/ folder. But I think this is a great start!

@maxbachmann
Copy link
Contributor

well we only need to check for assets anyways

@DO1JLR DO1JLR merged commit d2f0d96 into master Feb 15, 2019
@DO1JLR DO1JLR deleted the deploy_css branch February 15, 2019 18:40
wolf128058 pushed a commit that referenced this pull request Sep 24, 2019
* generate css with travis

Eventually this could work. We will see...

* sassc -> sass

* Travis need sudo

* use ruby-sass

* Use sass -t compress

This don't need a extra script for minify css

* delete old python script

* make deployment optional

* prepare travis switch to show this is live system

* new method including the css

Now lektor detects if the file is local available. If it is, it will use it.
If not, it will use the remote file and linking to the local font folders, because not every browser will accept the remote fonts that easy on localhost.

* Expand sass explaination

* improve lektor locale css config

lektor does not trigger a new build only if a file like main.min.css
suddenly appear or dissappear. It need a manual trigger like deleting
the cache or modify the templeates.

make will now delete the cache.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Please Review Please review this pull request
Projects
None yet
3 participants