Skip to content
This repository has been archived by the owner on Jan 23, 2018. It is now read-only.

Initial load can show all possible states #77

Open
skalnik opened this issue Dec 10, 2014 · 2 comments
Open

Initial load can show all possible states #77

skalnik opened this issue Dec 10, 2014 · 2 comments
Labels

Comments

@skalnik
Copy link
Owner

skalnik commented Dec 10, 2014

Due to the move to a completely javascript based front end, on slow internet (like conferences), for a second after load all states are visible. This means "sign in", "sign out", the table for the queue, and "Queue is empty" all appear. I think the initial load should do a server side render of the HTML so that your view is correct, and then we can ensure it auto-updates after that. May require moving from Knockout, not sure.

@skalnik skalnik added the bug label Dec 10, 2014
@noopkat
Copy link
Collaborator

noopkat commented Dec 10, 2014

agreed, and I’d like to move away from knockout if possible. It worked as a proof of concept, but I wasn’t 100% happy with it as an implementation.
In general - happy to drop client side refreshing altogether if you didn’t find it useful.

@skalnik
Copy link
Owner Author

skalnik commented Dec 10, 2014

In general - happy to drop client side refreshing altogether if you didn’t find it useful.

I think it was super cool and would like to keep it. But lets give it another pass and make it more robust. Great proof of concept 👍

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

No branches or pull requests

2 participants