Used this addon for this years Lunar Festival, and I found some issues and suggestions for improvement:

• Notes on each icon what the name of the NPC is, and for instances name the name of the instance
  • The second part is kind of crucial, as instances are often bunched up.
• Indoor and instance maps
  • Once you get inside the instance there is no indication where the NPC actually is.
  • As for indoor, a good example is the caves where Maraudon is inside, there are 3 entrances to this dungeon and an icon that points to the (correct) entrance to the dungeon would be very helpful.
• Icons are currently rendered under POI icons on the map, making it impossible to get the tooltip to show, and it's easy to miss one because it's covered by a town/city POI.
• ID 8647, add a note that you need to see Blasted Lands before the Draenor invasion, you do this with a buff
  • Preferably, add the NPC coordinates as well, or its own icon on the map (coords are 48.16, 7.29 by the way)
• Most icons for dungeons, if not all of them in my case, as well as some outside didn't get removed after finishing the quest, had to /reload to make them disappear

Lastly, this is probably a personal preference; add some rounded borders on the icons to make them look better and stand out more.