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.

Sign up for GitHub

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

Jump to bottom

dist includes files you don't need #43

Closed
anaisbetts opened this issue Oct 19, 2015 · 2 comments
Closed

dist includes files you don't need #43

anaisbetts opened this issue Oct 19, 2015 · 2 comments
Labels
grunt

Comments

@anaisbetts
Copy link

In Electron, there's no reason to minify files since you're shipping them in-box, or to include a bunch of different web-compatible fonts since you know you're operating against Chrome 45, you can strip these
@connors connors added the grunt label Oct 20, 2015
@connors
Copy link
Owner

connors commented Oct 31, 2015

💥 Good point. Just me thinking like a web dev. I'll clean things up.

@connors
Copy link
Owner

connors commented Oct 31, 2015

photon.min.css has been removed from /dist in 32a502c. The font files are in there for now since they're still need for the web docs.

I'll remove them from the download zip of photon for now though.

@connors connors closed this as completed Oct 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
grunt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@anaisbetts @connors