bug#1305: All code that currently beeps should use visual bell instead

[Eli Zaretskii]

> Date: Wed, 21 Apr 2021 16:01:23 +0000
> From: Gregory Heytings <gregory@heytings.org>
> Cc: Alan Third <alan@idiocy.org>, 1305@debbugs.gnu.org,
>  Michael Welsh Duggan <mwd@md5i.com>, Stefan Kangas <stefan@marxist.se>,
>  jasonspiro4@gmail.com, monnier@iro.umontreal.ca,
>  Lars Ingebrigtsen <larsi@gnus.org>
> 
> > Anyway, speaking about other faces you could inherit from (for the echo 
> > area flash), how about 'highlight' instead of 'match'?
> >
> > Seems more appropriate.
> 
> Yes, I noted that you'd prefer to use highlight instead of match, 
> presumably for a certain logical consistency.  If highlight had been 
> yellow, that's what I would have done.

IMO, we shouldn't use highlight for anything other than showing text
sensitive to mouse or to cursor entry.  That's what highlight was
invented for, and using the same colors for an utterly different
purpose is sub-optimal UI and UX.