chengming-1 / MI-449-js-collect-and-display-information

0 stars 0 forks source link

Project Feedback #1

Open chengming-1 opened 3 years ago

chengming-1 commented 3 years ago

Build a dating profile generator

@egillespie Can you take a look at this? It's hosted here and meets the following criteria:

egillespie commented 3 years ago

Your solution is looking really nice, Chengming! I like the function and variable that avoid duplicating a lot of code and can see the profile updating in real-time. Sweet! 🤘

There are two touch-ups I'd like you to make to the profile before I ship this. Here's a screenshot demonstrating both:

image

  1. Show the "Hi, my name is Erik Gillespie" part in <h1>..</h1> tags to match the provided template
  2. Replace EMAIL_ADDRESS and PHONE_NUMBER in the href attributes with the typed values

Comment back if you need tips or another review. You've got this! 🤘

chengming-1 commented 3 years ago

fixed

egillespie commented 3 years ago

Nice, the heading looks good now and I see some adjustment to the links. It appears that the href attributes are always set to mailto: and tel: though, instead of mailto:gille143@msu.edu and tel:555-1234 or whatever else I type in the email and phone fields.

Can you add the typed values to those href attributes?

chengming-1 commented 3 years ago

done

egillespie commented 3 years ago

Nailed it. Awesome! 🔨 :shipit: