I didn't mean to imply that you were an idiot, I apologize if it sounded that way.
I kinda thought that was what the last asterisks was for, thats why I left that one.
As for still getting the error, ensure you're not cached. I just tried it and its fine (granted, it won't work as anticipated and still has some warnings). To fix you're warnings, anytime you're retrieving user input, always check if its set, otherwise default it:
$myColor = '';
$myColor = $_GET['color'];
// Or, a little more elegant:
$myColor = isset($_GET['color']) ? $_GET['color'] : '';
Also, <select> doesn't take a value, instead, if the color is selected you add a 'selected = "selected"' to you're option tag:
<p>Select a color<br />
for ( $i = 0; $i < count( $rowcolors ); $i++ )
$sSelected = (isset($_GET['color']) && $_GET['color'] == $rowcolors[$i]) ? ' selected="selected"' : '';
echo "<option value=\"" . $rowcolors[$i] . $sSelected . "\">" . $rowcolors[$i] . "</option>\n";
I'll leave you to the table, thats the meat of this one.