AdamVig / goco-student

Gordon College Student app
https://gocostudent.adamvig.com
1 stars 0 forks source link

Allow for insertion of both info and interaction modules into main page #53

Closed AdamVig closed 9 years ago

AdamVig commented 9 years ago

Currently, there is an ng-repeat that iterates over the list of selected modules and, for each selected module, creates and <info-module> element that it populates with all of the data from the selected module.

To add interaction modules, there will have to be a check of what type of module the selected module is, then the view will have to conditionally render either an <info-module> or an <interaction-module>.

AdamVig commented 9 years ago

Alternatively, the current method of rendering modules in the main view could be kept in place and interaction modules could be rendered exclusively at the end of the list. Arguments for this:

Arguments against this:

Options:

  1. Allow interaction modules to be reordered with info modules so that the two types of module can be interspersed in the list.
  2. Separate info and interaction modules into two sections in the selection list so that the user can understand why the interaction modules show up at the bottom of the list.
AdamVig commented 9 years ago

The Solution

  1. Maintain the current module list view and selection options view.
  2. Combine infoModule and interactionModule directives into dataModule to allow for ng-repeat behavior in the main view.
  3. Standardize parameters on new hybrid module to allow for different behavior but consistent markup.