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

Added loading component #106

Conversation

lazywithclass
Copy link
Contributor

This is part of the task of taking care of TODOs as mentioned #64 (comment)

@lightandluck
Copy link
Collaborator

Thanks for getting to this! Please pull and create a new branch from our latest master. We've gone through a huge architectural change by removing Typescript. It may be easier to just redo your changes instead of merging or rebasing.

@lightandluck
Copy link
Collaborator

In main.html, there is a div.row that contains an old progress indicator. Please replace it with your new loading component for consistency. It looks great!

@lazywithclass
Copy link
Contributor Author

@lightandluck cool, thanks for the prompt review, I so love TypeScript removal :D

Will pull again tomorrow.

@lazywithclass lazywithclass deleted the adding-loading-component branch August 19, 2017 14:12
@lazywithclass
Copy link
Contributor Author

lazywithclass commented Aug 19, 2017

@lightandluck #108 :)

It was easy enough to just remove the old loader, as the new kicks in as soon as data is requested, let me know if it's ok.

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