submitting homework 4 #12

Merged
merged 1 commit into from Dec 21, 2018

Conversation

Projects
None yet
2 participants
@ackalil

ackalil commented Dec 17, 2018

I realize that the survey exercise is incomplete but I'm getting a cold and have a cloudy mind so I'm forcing myself to stop so I can get a decent night's sleep.

@vodnik

vodnik approved these changes Dec 21, 2018

You seemed comfortable with this after we discussed it in class, but please ping me on Slack or stop by during office hours if you have outstanding questions on any of the jQuery.


$button.on('click', function(event) {
event.preventDefault();
let madlib1 = $('h1').text('A startup that is ' + startupX[random1] + ', but for ' + startupY[random2]);

This comment has been minimized.

@vodnik

vodnik Dec 21, 2018

Owner

Good start!

// When the user adds the question to the survey, it should be automatically numbered and a text box should be added below it.
// Each survey question should also include a "Remove question" button that a user can click to remove the question from the survey.

function addNewQuestion($list, question) {

This comment has been minimized.

@vodnik

vodnik Dec 21, 2018

Owner

I like that you abstracted this into its own function -- it's really useful to be thinking about architecture and maintainability.

@vodnik vodnik merged commit 8bc9cec into vodnik:master Dec 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment