"Build a Weather App (2015)" was retired on June 6, 2018. You are now viewing the recommended replacement.

Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript Regular Expressions in JavaScript Validating a Form Form Validation

In Challenge 1, I'm getting an error 'The hexRegEx variable does not hold a regular expression'.

The function is isValidHex. Any help is appreciated.

app.js
// Type inside this function
function isValidHex(text) {
  var RegEx = /^#[\da-z]{6}$/i;
  var hexRegEx = text.match(/^#[\da-z]{6}$/i);
}

const hex = document.getElementById("hex");
const body = document.getElementsByTagName("body")[0];

hex.addEventListener("input", e => {
  const text = e.target.value;
  const valid = isValidHex(text);
  if (valid) {
    body.style.backgroundColor = "rgb(176, 208, 168)";
  } else {
    body.style.backgroundColor = "rgb(189, 86, 86)";
  }
});
index.html
<!DOCTYPE html>
<html>

<head>
    <title>DOM Manipulation</title>
</head>
<link rel="stylesheet" href="style.css" />

<body>
    <div id="content">
        <p>Enter a valid hex value below to make the screen turn green.</p>
        <input type="text" id="hex" name="text">
    </div>
    <script src="app.js"></script>
</body>

</html>

2 Answers

Sean T. Unwin
Sean T. Unwin
28,690 Points

Your RegEx variable's value is correct. Rename RegEx to hexRegEx (remove the old hexRegEx altogether) and Challenge 1 should pass.

Challenge 2 requires the use of test().

Thanks, Sean! I was overthinking this and trying to place the value of text into hexRegEx.