I think you've done a good job on this map, MPG, but it's a bit hard to comment when there's very little information about what's where. That green island in the middle, for example - are those trees? Also, what are the dark brown districts?
Some things that stand out straight away to me:
--Is there a land-route into the city and, if so, where is it?
--There are a few places where the roads clearly go over the top of the building outlines. I think it might look better if you remove the road layers altogether, and allow the thicker spaces between buildings to indicate what is public space and what is built space.
--I really like that you have incorporated lots of courtyards and alleyways into your city. You could try to add some buildings *within* those courtyards, as that would often be the case in historic cities
Hope some of that is helpful
Wingshaw