I think _not_ closing in that case was something NJ cited as a bug when we walked through Glenn's original implementation during sprint planning.  It feels weird to have the popover superimposed on top of the inline color editor (even if it's easy to dismiss).