Currently, if the logic flow enters the "else" case here, there would be an ending ul tag without an associated start tag.  I think this end ul tag should be placed outside of the conditional block to match up with the suggestion of placing the start ul tag outside of the conditional block.