Can someone please help me out with this problem. I don't understand what I am doing wrong.

Question

It is saying that my variable isn't holding a regex expression.

app.js

// Type inside this function
function isValidHex(text) {
  const hexRegEx = /^#[A-F][a-f][0-7]$/i.test(text);
}

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">
    </div>
    <script src="app.js"></script>
</body>

</html>

Answer 1 · 2019-05-27T00:16:02Z

May 27, 2019 12:16am

They want you to just start by assigning a regular expression to the variable hexRegEx. Don't call the test() method on it yet. Your code as it is right now will actually assign a boolean value to hexRegEx, the result of calling test().

While we're at it, there are a few other things your regex will need to do to meet the requirements:

the letters and numbers will not necessarily be in this order
there will be exactly 6 characters after the # symbol
we need to include the digits 8 & 9 as well as 0-7
FYI if you're using the case insensitive flag, you don't need to also check for both a-f and A-F

Let me know if you have more questions.

Welcome to the Treehouse Community

Looking to learn something new?

Darrin Spell Jr

Darrin Spell Jr

Can someone please help me out with this problem. I don't understand what I am doing wrong.

1 Answer

Brendan Whiting

Brendan Whiting