charwking / movie-club

A web app for running a work movie club
MIT License
3 stars 5 forks source link

Clean up Dashboard and change directory structure #57

Closed charwking closed 9 years ago

charwking commented 9 years ago

@ttonyan10 -- I'd like to apply some cleanups to the dashboard, similar to the ones I applied to the authentication related pages, but don't want to create too many merge conflicts for you. I'll work on other things and plan on applying them whenever you're at a good stopping point. (No rush.)

Also, I was thinking about changing the directory structure to something a little more organized. Is that okay with you? I was thinking something below. The benefits I see are that it clearly separates the pages from the partials and libs, and embeds page-specific less files in the same directory as the related html file.

- src
   + app
   + images
   - libs
      - auth
         - auth.module.js
         - authApi.service.js
      + core
      + properties
      + users
   - pages
      - dashboard
         - dashboard.controller.js
         - dashboard.html
         - dashboard.less
         - dashboard.module.js
         - dashboard.routes.js
      + login
      + logout
      + register
      + user-movies
   - partials
      + nav
   - styles
      - common.less
      - mixins.less
   - index.html
charwking commented 9 years ago

I cleaned up the dashboard. Let's leave the directory structure as-is for now.