besser lesbarer, berichtigter Code

[MudGuard]

Hi,

for ($rr = 255; $rr >= 0; $rr = $rr-15) {
          if (strtoupper(str_pad(dechex($rr),2,"0",STR_PAD_LEFT)) === "$r") {
            $contrast = 255-$rr;
            echo("<tr>
                    <td id="x"

Wie oft darf eine eindeutige id in einem Dokument vorkommen?

style="border:2px solid #".str_pad(dechex($contrast),2,"0",STR_PAD_LEFT)."FFFF;
                                width:6px;
                                background-color:#".str_pad(dechex($rr),2,"0",STR_PAD_LEFT)."0000">

<a style="font-size:10px"

Warum arbeitest Du hier mit style-Attributen? Das bläht den Code in überflüssiger Weise auf, zentrale Formate sind besser geeignet.
Daß Du die Farbe mit dem style-Attribut setzt, ist ja noch einzusehen, aber alles, was für alle Zellen gleich ist, sollte zentral geregelt werden.

href="admin.php?c=$c&r=$r">
                        &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

urgs. display:block und width/height im zentralen CSS setzen, dafür die nbsps wegwerfen.

Zum Auffinden weiterer Fehler: den erzeugten HTML-Code mal durch den Validator jagen ...

cu,
Andreas