supongo que estas haciendo un bucle y para eso el <span> debería de quedar fuera puesto que hasta un novato como yo (como tu dices) sabe que no se puede intentar cerrar una etiqueta fuera de la que lo encierra (en este caso el <span></span>), por lo tanto el </li> y el </ul> deberían de ponerse así:
<ul class="categoryitems"><li></li></ul>
pero no tendría sentido hacer eso ya que no hay nada a lo que se le pueda asignar el estilo de lista.
ojala y entiendas que este foro es para apoyarse unos a otros y no para criticar.
PD. no se si es la primera vez que respondo directamente a un post tuyo pero de algo estoy seguro, esta es la última.