Commit 918debb4 authored by Jakob Moser's avatar Jakob Moser
Browse files

Remove additional exerciseIndex parameter

parent 5e59490d

static/js/exercises.cards.mjs

+3 −4
Changes for static/js/exercises.cards.mjs: 3 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -55,14 +55,13 @@ function createExerciseCardActionButtons(manuallyConfirm) { 
/**

 * Create a DOM card for the given exercise

 * @param {Exercise} exercise The exercise

 * @param {Number} exerciseIndex The index of the exercise

 * @param {Function} onTitleClick Event handler for click on the title

 */

export function createExerciseCard(exercise, exerciseIndex, onTitleClick) {

    const cardEl = createCard(exercise.name, "#" + exerciseIndex)

export function createExerciseCard(exercise, onTitleClick) {

    const cardEl = createCard(exercise.name, "#" + exercise.index)



    cardEl.classList.add("exercise")

    cardEl.dataset.exerciseIndex = exerciseIndex

    cardEl.dataset.exerciseIndex = exercise.index

    // Add Bootstrap Icon's 'bi' class to the heading, because it will apply styles to

    // the h2::before element, whose content we set automatically in CSS, depending on the

    // solved state of the exercise

static/js/yalikejazz.mjs

+5 −6
Changes for static/js/yalikejazz.mjs: 5 added lines, 6 removed lines.
Original line number Diff line number Diff line
@@ -19,18 +19,17 @@ function findNextUnsolvedExerciseIndex(currentExerciseIndex) { 
/**

 * Create a DOM card for the given exercise, displays is as solved if solved and inits event listeners

 * @param {Exercise} exercise The exercise

 * @param {Number} exerciseIndex The index of the exercise

 * @returns The DOM card

 */

function createAndInitExerciseCard(exercise, exerciseIndex) {

    const cardEl = createExerciseCard(exercise, exerciseIndex, () => {

        if (!currentState.solvedExercises.includes(exerciseIndex) && currentState.exerciseIndex !== exerciseIndex) {

function createAndInitExerciseCard(exercise) {

    const cardEl = createExerciseCard(exercise, () => {

        if (!currentState.solvedExercises.includes(exercise.index) && currentState.exerciseIndex !== exercise.index) {

            // Only skip to exercises that are not solved and aren't itself

            startExercise(exerciseIndex)

            startExercise(exercise.index)

        }

    })



    if (currentState.solvedExercises.includes(exerciseIndex)) {

    if (currentState.solvedExercises.includes(exercise.index)) {

        displayAsSolved(cardEl)

    }