microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
162.19k stars 28.55k forks source link

Folded region audio cue sound too harsh #142016

Closed miguelsolorio closed 2 years ago

miguelsolorio commented 2 years ago

I've been using audioCues.enabled for the past few weeks, even though I may not be the primary audience for this, and I've really enjoyed the added sounds as it enhances my development workflow.

However, I've found that the folded region audio cue sounds much too harsh compared to errors/breakpoints. To me, it sounds like someone fumbling through papers or writing on something and in the beginning this felt like I was still in a meeting 😄 see sample below (you'll need to turn on the 🔊 sound in the video clip):

https://user-images.githubusercontent.com/35271042/152199028-e22c15f1-b6ea-4417-a3c4-f87e892affea.mp4

Comparing the different sounds, the folded cue stands out by a lot and don't think it is warranted given the importance compared to errors/breakpoints. I think the sound should be more gentle and less harsh as it is less important. As I said, I may not be the primary user for this but just my feedback on the sound.

cc @gino-scarpino

isidorn commented 2 years ago

@misolori awesome that you are selfhosting on this 👏 I agree that the folded sound can be irritating to the ear, and that it could be more pleasant.

Let's assign this to February as we plan to polish sounds and @gino-scarpino can think about this feedback.

gino-scarpino commented 2 years ago

Great feedback and ear @misolori. We were totally exploring a paper sound here, but we need to re-consider. I agree, especially on repetition, it's harsh. Align more with the other sounds. Added to my list.

Thank you!


From: Isidor Nikolic @.> Sent: Wednesday, February 2, 2022 8:56 AM To: microsoft/vscode @.> Cc: Gino Scarpino @.>; Mention @.> Subject: Re: [microsoft/vscode] Folded region audio cue sound too harsh (Issue #142016)

@misolorihttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmisolori&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227471799%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=FWJUZBU3RJpj6qW8EQ%2F26IflCjfFfDY0BVe7haU64zA%3D&reserved=0 awesome that you are selfhosting on this 👏 I agree that the folded sound can be irritating to the ear, and that it could be more pleasant.

Let's assign this to February as we plan to polish sounds and @gino-scarpinohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgino-scarpino&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227471799%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=2%2Fb9RcpLea5XIr4M3xgU0gDL63IZTFTd9KEhlA55uWc%3D&reserved=0 can think about this feedback.

— Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fvscode%2Fissues%2F142016%23issuecomment-1028146816&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227522232%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=bP%2FPQjVxU8YAwJaJhv10EbgY1U1nIUsK6NV1pAsDv1g%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAWWW7HTIWDKYLGKDIG4XETDUZFO5XANCNFSM5NMSSXMA&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227522232%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=5NqV8JfVPKXyOWtLFR1328W9fybPs01xy%2Fwi0niq6zA%3D&reserved=0. Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227522232%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=OSaG79Hj66UPMqrK6SFtTNwuu4CypdIaWv9zJz6wRt4%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04%7C01%7Cginoscarpino%40microsoft.com%7Cff57ea4aa59e4099c97e08d9e66d074b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637794178227522232%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=zSLoNyB65z5CwdHJVGUkvTyerJ2atBi%2FXLC0ceChQtA%3D&reserved=0. You are receiving this because you were mentioned.Message ID: @.***>

boltex commented 2 years ago

I cant seem to find the settings to choose a particular .wav file for which trigger, let alone the list of triggers...

isidorn commented 2 years ago

@boltex not supported yet.

isidorn commented 2 years ago

@gino-scarpino provided a new sound and @hediet added it to vscode, thus closing this as fixed 👏

steverep commented 2 years ago

I understand this issue is closed, but I now have the opposite issue. I find the new sound barely audible and with my screen reader on it's not even noticeable. Maybe it's my speakers or my ears, but the new sound in 1.65 is useless to me. If I'm skimming down code quickly with my screen reader, I don't even notice this new one relative to the speech.

When evaluating something as subjective as "harshness" of a sound, please keep in mind how relative that is, not just to the user but also to the environment (e.g. speaker specs and location, other background sound like a screen reader or music, etc.).

As I suggested in another issue, priority should be given to developing user choice here. Make a bank of a bunch of sounds available, choose some suitable defaults, but there needs to be a user setting for each cue.

That said, audio cues in general are extremely helpful to me as a screen reader user so thanks for putting in the effort.

isidorn commented 2 years ago

@steverep thank you very much for your feedback, this is extremely valuable to us.

We agree that sound preference is something personal, so we might need to add more customisation for this in the future. You are suggesting customisation of each sound. While that might come in the future we will first introduce a control for the audio cue volume. I will let you know once we have that so you can try it out and let us know if this has helped in this particular case.

Looking forward to more feedback.

cannona commented 2 years ago

Ouch! @steverep is absolutely right here. I came here to file a bug that the folded code sound was not working at all. Turns out, I just had to turn off the speech of my screen reader to hear it.

I would argue that this sound should be more harsh, or at least more prominent than other sounds. This sound is the only reliable way for screen reader users to know that there is some folded code at the cursor. This might not sound like a big deal, until you realize what confusion might be caused by thinking that part of your code is missing, and not just hidden.

Needless to say, I'm pretty disappointed with this change. I realize that most of us are volunteers, but if at all possible, I would encourage the team to find ways to seek feedback from people who are members of the target audience for accessibility features before making changes like this in future. I would very much appreciate having the original behavior reinstated if there is no better solution for this in the short term.

Bug again, like @steverep, I am very grateful for this functionality, and for all the amazing efforts from the team relating to accessibility. In spite of the occasional complaint, such as the above, this application has truly changed my life as a developer. Thank you all!

steverep commented 2 years ago

We agree that sound preference is something personal, so we might need to add more customisation for this in the future. You are suggesting customisation of each sound. While that might come in the future we will first introduce a control for the audio cue volume. I will let you know once we have that so you can try it out and let us know if this has helped in this particular case.

More customization is always welcome, but I think volume is unlikely to make much difference in this case. As I and @cannona said, if screen reader users are the target demographic for these cues, then the chosen sounds need to stand out relative to fast speech right before and right after the sound. This new 1.65 sound would probably have to be near a dangerous volume to do that.

isidorn commented 2 years ago

@cannona @steverep great feedback! Thanks a lot. Let me address your comment regarding encouraging team to find feedback. I think we are actually not so bad at this since a lot of our screen reader users are using VS Code insiders and we get feedback constantly before something gets pushed to stable. Having said that, this one somehow slipped and we plan to fine tune the sound to be something more like the previous one. I have created this issue for this https://github.com/microsoft/vscode/issues/144927

And I suggest that you switch to using VS Code insiders once we push the fix next week so you get unblocked here. Also consider to join our Accessibility gitter channel where we do some discussions and announcments https://gitter.im/Microsoft/vscode-a11y

Thanks

cannona commented 2 years ago

I think we are actually not so bad at this since a lot of our screen reader users are using VS Code insiders and we get feedback constantly before something gets pushed to stable.

That's totally fair, and my words in this regard were, I think, poorly considered. You folks are far ahead of the vast majority of projects out there, and I should have taken greater care to not suggest otherwise.

Thank you again!

isidorn commented 2 years ago

@cannona thank you very much for the feedback and please keep it coming :)

isidorn commented 2 years ago

@steverep @cannona we are doing a survey to better understand our users and to design better default sounds. We would like to know if you are using headphones or speakers when working with VS Code?

Also I have opened this issue for discussion on sound customisations (nothing on the plan now, but we might tackle it in the future): https://github.com/microsoft/vscode/issues/145204

We are also working on the new folded sound that will most likely be in VS Code Insiders tomorrow - so we would really appreciate if you try it out and provide feedback so we can fine tune it more before we ship it in stable.

cannona commented 2 years ago

@isidorn I WFH, so mostly I'm using speakers. Also, audio folks have a way to measure the relative loudness of a sound. It's known as LUFS. You might look into it, as it may help you to balance things a bit better. As I understand it, it takes into account the average loudness of a sound, unlike most calculations, which look only at the peaks.

Thanks for the info.

steverep commented 2 years ago

@isidorn I use both - speakers in my home office and mostly headphones elsewhere for privacy.

Adding to @cannona's recommendation to look at LUFS,you can easily alter this using Audacity. Here's the manual page for loudness normalization.

You could also go deeper and read up on psycho-acoustics 📚 🔉 🥼 .