Skip to content
This repository was archived by the owner on Jun 7, 2023. It is now read-only.

Listeners should be correctly attached to timed elements now #173

Merged
bnmnetp merged 1 commit into from
Jul 5, 2016
Merged

Listeners should be correctly attached to timed elements now #173

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 1 addition & 6 deletions runestone/assess/js/timed.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ Timed.prototype.init = function (opts) {
this.incorrectStr = "";
this.skippedStr = "";
this.skipped = 0;
this.hasRenderedFirstQuestion = false;

this.currentQuestionIndex = 0; // Which question is currently displaying on the page
this.renderedQuestionArray = []; // list of all problems
Expand Down Expand Up @@ -344,12 +343,8 @@ Timed.prototype.renderTimedQuestion = function () {
// If the timed component has listeners, those might need to be reinitialized
// This flag will only be set in the elements that need it--it will be undefined in the others and thus evaluate to false
if (this.renderedQuestionArray[this.currentQuestionIndex].needsReinitialization) {
// if this is the first time we're rendering the first question, nothing should be reinitialized
if (this.currentQuestionIndex !== 0 || this.hasRenderedFirstQuestion) {
this.renderedQuestionArray[this.currentQuestionIndex].reinitializeListeners();
}
this.renderedQuestionArray[this.currentQuestionIndex].reinitializeListeners();
}
this.hasRenderedFirstQuestion = true;
};


Expand Down