add_element should be private
clean up Eml.Markup.to_attr_value and make it behave more consistent