What is his problem please?

Tell us what’s happening:

Your code so far


<h2>CatPhotoApp</h2>
<main>
 <p>Click here to view more <a href="#">cat photos</a>.</p>
 
 <a href="#"><img src="https://bit.ly/fcc-relaxing-cat" alt="A cute orange cat lying on its back."></a>
 
 <p>Things cats love:</p>
 <ul>
   <li>cat nip</li>
   <li>laser pointers</li>
   <li>lasagna</li>
 </ul>
 <p>Top 3 things cats hate:</p>
 <ol>
   <li>flea treatment</li>
   <li>thunder</li>
   <li>other cats</li>
 </ol>
 <form action="/submit-cat-photo">
   <input id="indoor" type="radio" name="indoor-outdoor">
   <label for="indoor">Indoor</label>
   <input id="outdoor" type="radio" name="indoor-outdoor">
   <label for="outdoor">Outdoor</label>
   
   <input type="text" placeholder="cat photo URL" required>    
   <button type="submit">Submit</button>
 </form>
</main>
<!-- 
Add a pair of radio buttons to your form. One should have the option of indoor and the other should have the option of outdoor. Both should share the name attribute of indoor-outdoor.
-->

link to the challenge?

Hello @mahmoudriad!

It seems that the problem is that you are not providing a value to your radio <input> element.

I see you have added the id attribute, but it does not set the value for that input, it is the value attribute that does it.

You can check the reference for Input Radio here and check an example on how to use it:
https://www.w3schools.com/tags/att_input_type_radio.asp

Hope it helps!

This is the challenge you are on, right?

You need to nest the radio inputs inside the labels, look at the example code given for how to nest the inputs.

<label for="indoor"> 
  <input id="indoor" type="radio" name="indoor-outdoor">Indoor 
</label>