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 trialscjprice
670 PointsIn Challenge 1, I'm getting an error 'The hexRegEx variable does not hold a regular expression'.
The function is isValidHex. Any help is appreciated.
// 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)";
}
});
<!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
28,690 PointsYour 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().
Andrii Kodola
Full Stack JavaScript Techdegree Student 16,870 PointsHey,
plus to what has been said, you need to return something in isValidHex().
scjprice
670 Pointsscjprice
670 PointsThanks, Sean! I was overthinking this and trying to place the value of text into hexRegEx.