code cleanup

This commit is contained in:
Jason White 2015-10-24 13:15:14 -05:00
parent fc2360b49b
commit ab29afec3c
4 changed files with 8 additions and 6 deletions

View File

@ -88,19 +88,18 @@ define(['jquery',
this.onContentLoaded = function(loadHelps) {
this.lessonInfoModel = new LessonInfoModel();
this.listenTo(this.lessonInfoModel,'info:loaded',this.onInfoLoaded); //TODO onInfoLoaded function to handle title view and helpview
this.listenTo(this.lessonInfoModel,'info:loaded',this.onInfoLoaded);
if (loadHelps) {
this.helpControlsView = null;
this.lessonView.model = this.lessonContent;
this.lessonView.render();
//load title view (initially hidden) << //TODO: currently handled via menu click but need to be able to handle via routed request
this.planView = new PlanView();
this.solutionView = new SolutionView();
this.sourceView = new SourceView();
this.lessonHintView = new HintView();
this.cookieView = new CookieView();
// parameter model & view
//TODO: instantiate model with values (not sure why was not working before)
var paramModel = new ParamModel({});
paramModel.set('screenParam',this.lessonContent.get('screenParam'));

View File

@ -13,9 +13,8 @@ define(['jquery',
},
infoLoaded: function(data) {
console.log (data);
this.trigger('info:loaded',this,data);
}
});
});
});6

View File

@ -10,15 +10,18 @@ define(['jquery',
JQueryForm) {
return Backbone.View.extend({
el:'#lesson-content-wrapper', //TODO << get this fixed up in DOM
initialize: function(options) {
options = options || {};
},
render: function() {
this.$el.html(this.model.get('content'));
this.makeFormsAjax();
this.ajaxifyAttackHref();
$(window).scrollTop(0); //work-around til we get the scroll down sorted out
},
//TODO: reimplement this in custom fashion maybe?
makeFormsAjax: function () {
var options = {

View File

@ -4,6 +4,7 @@ define(['jquery',
function($,_,Backbone) {
return Backbone.View.extend({
el:'#header #lesson-title-wrapper',
render:function(title) {
var lessonTitleEl = $('<h1>',{id:'lesson-title',text:title});
this.$el.html(lessonTitleEl);