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

improvements and new example #29

Merged
merged 4 commits into from
May 15, 2017
Merged

improvements and new example #29

merged 4 commits into from
May 15, 2017

Conversation

anoymouserver
Copy link
Contributor

  • JavaScript code is now linted with JavaScript Style Guide
  • moved documentation to markdown
  • added example6 with Bootstrap Design

anoymouserver and others added 4 commits May 9, 2017 23:11
@anoymouserver
unify and beautify code
60a314f 
add editorconfig
@anoymouserver
move documentation to markdown
98803b8 
- its easier to edit
- you will see the diff if some changes occur
@anoymouserver
update example5 to CDN, add example6 with Bootstrap design
849a893 
(Bootstrap currently at v4.0.0-alpha.6)
@anoymouserver
compress .min.js better
ebcb4d0
@adolfintel
Copy link
Member

Thanks man! Looks good ;) Doesn't break anything.
Before I merge it, I have a question to ask: what command did you use to minify it? You just said "better"

@anoymouserver
Copy link
Contributor Author

I also used uglifyjs but for my first commit I failed while compressing and the file was twice as large..

@adolfintel
Copy link
Member

Oh, ok. I'll merge this when I get back from work.
Thank you

@adolfintel adolfintel merged commit 3f06ab1 into librespeed:master May 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anoymouserver @adolfintel