La mezcla de contenido HTML / PHP abriendo y cerrando bloques produce un código verdaderamente horrible y difícil de analizar o de depurar.

Para casos como el tuyo, puede ser interesante el uso de [`printf()`][1] o de [`sprintf()`][2]. Funcionan igual, con la diferencia de que con la primera imprimes in situ, y con la segunda almacenas el contenido en variables, por ejemplo si estás concatenando o creando un array o un objeto.

Así en lugar de esto:

    <a href="mostrar_notarias.php?documento_seleccionado=<?php echo $documento_seleccionado; ?>&nombre=<?php echo $nombre; ?>">Continuar</a></button>

Tendríamos:

    $query = array(
                    'documento_seleccionado'=>$documento_seleccionado,
                    'nombre'=>$nombre
                  );

    printf('<button>
              <a href="mostrar_notarias.php?%s">Continuar</a>
            </button>',
            http_build_query($query));

He agregado la etiqueta `<button>` de apertura, la cual no aparece.

Y en lugar de:

    echo '<td><a href="mostrar_horas.php?documento_seleccionado=' . $fila['id'] . '&nombre=' . $fila['nombre'] . '&notaria_seleccionada=' . $fila['id'] . '" class="btn btn-outline-warning"><h6>Continuar</h6></a></td>';

Tendríamos:

    $query = array(
                    'documento_seleccionado'=>$fila['id'],
                    'nombre'=>$fila['nombre'],
                    'notaria_seleccionada'=>$fila['id']
                  );

    printf('<td>
             <a href="mostrar_horas.php?%s" class="btn btn-outline-warning"><h6>Continuar</h6></a>
            </td>',
            http_build_query($query));

Como vez, tanto `printf()` como `sprintf()` funcionan poniendo marcadores donde va cada dato, y al final pasas los datos en la posición en que deban ir.

En cuanto a tu código PHP, podemos darle claridad usando el operador de fusión de null (PHP 7 o superior) o bien operadores ternarios en su defecto.

    <?php
    
        $doc = $_GET['documento_seleccionado'] ?? NULL;
        $nombre = $_GET['nombre'] ?? NULL;
        $id = $_GET['notaria_seleccionada'] ?? NULL;
    
        if ($doc && $nombre && $id) {
        
            // Conecta a la base de datos
            $pdo = new PDO('mysql:host=localhost;dbname=notaria', 'root', '');
            
            // Recoge el ID del documento seleccionado del formulario
            
            // Ejecuta una consulta para obtener los datos del documento seleccionado
            $stmt = $pdo->prepare('SELECT * FROM notarias WHERE id = :id');
            $stmt->bindParam(':id', $id, PDO::PARAM_INT);
            $stmt->execute();
            $documento = $stmt->fetch();      
        }
    
    ?>

Es un código más limpio. Te aconsejo que evites el uso de variables con nombres kilométricos. Tal es el caso de `$documento_seleccionado`, con nombrarla `$doc` se salva la comprensión del código y se gana mucho en legibilidad. Le puedes llamar `$pepitoDeLosPalotesYMasCosas` o lo que quieras, pero demasiada verbosidad innecesaria hace que el código sea infumable muchas veces. 


  [1]: https://www.php.net/manual/es/function.printf.php
  [2]: https://www.php.net/manual/es/function.sprintf.php