Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Play a sound for Granite.MessageDialog #471

Open
hanaral opened this issue Mar 2, 2021 · 3 comments
Open

Play a sound for Granite.MessageDialog #471

hanaral opened this issue Mar 2, 2021 · 3 comments
Labels
Needs Design Waiting for input from the UX team Priority: Wishlist Not a bug; but a new feature or enhancement

Comments

@hanaral
Copy link

hanaral commented Mar 2, 2021

Problem

Many users may react slowly or not be able to immediately notice an error popup. The user has to be staring at their to notice a popup, which is pretty bad if it is a critical error.

Proposal

Gala should play a crisp 'dunn' sound effect when a Granite.MessageDialogue with an error appears, and for when a Granite error views appears.
This should be done on an opt-in basis to avoid accidentally confusing users.
I'd also suggest adding a lighter sound effect for authentication popups.

tl;dr
Error views and granite error dialogues should have unique audio cues

Prior Art

Maybe I'll add some later

@danirabbit danirabbit transferred this issue from elementary/gala Mar 2, 2021
@danirabbit
Copy link
Member

Since this would need context, it would have to happen in Granite. The Window Manager isn't aware of the dialog's context, just that it's a dialog.

This would be hard to enforce consistently since apps may choose not to use Granite.MessageDialog to show errors

@danirabbit danirabbit added Needs Design Waiting for input from the UX team Priority: Wishlist Not a bug; but a new feature or enhancement labels Mar 2, 2021
@cassidyjames
Copy link
Contributor

I’m pretty 👎 about dialogs playing sounds in general; notifications are the platform way to notify for background tasks, and dialogs generally assume the user is present. I believe GNOME used to (optionally) play an FD.o sound when dialogs were thrown, but it was removed because it was annoying. It's probably worthwhile to investigate when and why they removed it if you want to petition for it to be re-added.

@hanaral
Copy link
Author

hanaral commented Mar 2, 2021

This isn't about notifying the user about a backrgound activity, you've already pointed out that the general way to notify is through notifications.
This is supposed to be a way for there to automatically be an auditory cue to fit with something of importance on screen - yes, I could use a screen reader, but what happens to the using accessibility features as features?
I think that this should just be something the user can disable like window animations.

Also is there no way for granite.messagedialog to emit a dbus signal or something that gala or even the notification server(which already plays sounds) can catch?
Edit: Also the issue of developers not using granite.messagedialog is their own problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Waiting for input from the UX team Priority: Wishlist Not a bug; but a new feature or enhancement
Projects
None yet
Development

No branches or pull requests

3 participants