While not as ostentatiously as big or cool as it could be, I find it to be fine. As long as it's the biggest elf npc city, and the most powerful, I will be fine with it. Though, it is lacking something special in it. Dagbor has the river and the fancy double wall thing, Makkada has interlocking wall segments with its back to the mountains, while Illas Tiltos just has just a double layer of walls.
Picture made by Aametherar the orc (Elves are the best!)