Terminal's "audible bell" option doesn't work on Windows desktop

gtritchie commented 5 years ago

System details

RStudio Edition : Desktop
RStudio Version : 1.3.421
OS Version      : Windows 10 1903 build 18362.329
R Version       : any

Steps to reproduce the problem

start desktop IDE
tools / global options / terminal
uncheck "audible bell"
start a terminal (any shell)
backspace at prompt

Describe the problem in detail

get a nasty Windows alert sound

Describe the behavior you expected

pristine silence, minus the faint clacking of finger on backspace key and the sounds of happy children playing in the distance

This is not new to 1.3; happens in 1.2 as well (didn't try 1.1). Almost certainly due to WinPty's hidden console: https://github.com/rprichard/winpty/issues/147

If so, may not be fixable in which case might want to hide this option on Windows. Probably controllable if we used Windows-10 1903's native PTY support.

Should also check VSCode's terminal (on a pre 1903 Windows-10) and see if they were able to override this.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs, per https://github.com/rstudio/rstudio/wiki/Issue-Grooming. Thank you for your contributions.

stale[bot] commented 3 years ago

This issue has been automatically closed due to inactivity.

rstudio / rstudio