Buenas tardes, creo que está todo bien
Hay algo un tanto extraño en cómo estás generando el código. Estás usando un solo punto de apertura y cierre de código php e incrustando el html a través de instrucciones echo. Esto se puede hacer así pero en general diría que es más frecuente trabajar con el código html libre dentro del código y abrir y cerrar código php cada vez que es necesario.
La idea es similar a esto:
<html>
<head>
<title>Prueba de PHP</title>
</head>
<body>
<?php echo '<p>Hola Mundo</p>'; ?>
</body>
</html>
Esto es un ejemplo tonto, ahí realmente no hace falta abrir php para mostrar Hola mundo, pero la idea es que tienes el código HTML y en determinados puntos abres etiquetas php para introducir código dinámico. El código estático quedaría fuera de php.
PHP es bastante versátil y te permite hacer las cosas de muchas maneras. Con el tiempo podrás irlo trabajando como te sea más cómodo. o incluso combinar estilos de trabajo con él.
Saludos