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

Small optimizations (development, README, typings, rxjs) & forms API #3

Merged
merged 5 commits into from
Jun 28, 2016

Conversation

TobiasKrogh
Copy link

@TobiasKrogh TobiasKrogh commented Jun 28, 2016

see commit list (basically just some refinements and improvements) and finally changed the slider to the new forms api

EDIT: if you want I can also spit into smaller Pull Requests but I thought it might be ok since having individual commits

@TobiasKrogh
Copy link
Author

I somehow did not manage to update the demo submodule content. Maybe you could include it directly without the submodule and ignore it upon updating the npm package... anyway: I did update the demo content and tested the new code, which works fine.

Find attached the demo content.
demo.zip

Cheers

@TobiasKrogh TobiasKrogh changed the title New forms api Small optimizations (development, README, typings, rxjs) & forms API Jun 28, 2016
@tb
Copy link
Owner

tb commented Jun 28, 2016

@TobiasKrogh thanks! I will check it evening and will handle the demo update.

@TobiasKrogh
Copy link
Author

@tb great, thank you 👍

@tb tb merged commit 4f6db0c into tb:master Jun 28, 2016
@TobiasKrogh TobiasKrogh deleted the new_forms_api branch June 28, 2016 20:47
@TobiasKrogh
Copy link
Author

Thank you for handling it so quick :) works like a charm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants