More refactoring.
Review Request #7484 — Created July 3, 2015 and discarded
This change accomplishes several things:
- Use a<nav>element for the header links.
- Change the frontend code to delay setting up the routes until the basic
elements of the page are fetched from the server.
- Added a spinner while the main data is loading.
- Added the admin link (which currently doesn't do anything, but that's next
up).
- Changed various URLs so that endpoints that return JSON are all under /api/
- Indentation fixes.
Verified that everything still works as expected.
Review request changed
- Change Summary:
-
Add some additional code that needed to be there (especially the
isAdminmethod), plus fix a couple bugs. - Commit:
-
6988894e1635652222b7eda789b944c21455966e9bb70e6bfe023d330f2f96adbb00aec4da3c5d00
-
Tool: PEP8 Style Checker Ignored Files: lib/frontend/x-student-view.js lib/frontend/main.js views/login.handlebars lib/schema.js lib/frontend/router.js css/style.less lib/routes.js views/layouts/main.handlebars Tool: Pyflakes Ignored Files: lib/frontend/x-student-view.js lib/frontend/main.js views/login.handlebars lib/schema.js lib/frontend/router.js css/style.less lib/routes.js views/layouts/main.handlebars
Tool: PEP8 Style Checker Ignored Files: lib/frontend/x-student-view.js lib/frontend/main.js views/login.handlebars lib/frontend/router.js css/style.less lib/routes.js views/layouts/main.handlebars Tool: Pyflakes Ignored Files: lib/frontend/x-student-view.js lib/frontend/main.js views/login.handlebars lib/frontend/router.js css/style.less lib/routes.js views/layouts/main.handlebars