With the changes being introduced by #296, we are only going to have either a elements or removeElements list. Modifier methods like allowElement modify the currently existing one.
I am wondering what we should do about cases like this:
let s = new Sanitizer({ removeElements: ["a", "p"] })
s.allowElement({ name: "a", removeAttributes: ["href"] })
With the current proposed behavior in #296, we would simply delete the "a" element from the removeElements list, but completely drop removeAttributes: ["href"]. Of course we don't have local attributes lists for anything but elements, so there isn't really anything we can do. But it does seem dangerous to me to just silently drop it (e.g. the link is unexpectedly clickable). Should we do something about this? I know we tried not to throw for any of the modifier methods. Should we at least have some suggestion about producing a console warning? Or maybe something else.