Use alert sound device, not default sound output device, for Growl sounds

fastsupply / growl

Automatically exported from code.google.com/p/growl

0 stars 0 forks source link

Use alert sound device, not default sound output device, for Growl sounds #145

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What feature or enhancement do you propose?
Switch Growl's sound output to go to the alert sound output device, not the 
default sound output device.

What problem does it solve/What benefit does it provide?
Growl notification sounds are closer in purpose to alert sounds than general 
sound playback, so the alert sound device is the more proper device to use.

Original issue reported on code.google.com by prhgr...@gmail.com on 30 Jun 2010 at 5:45

GoogleCodeExporter commented 9 years ago

One might think we could do this by using AlertSoundPlayCustomSound instead of 
SystemSoundPlay, but that would be worse: While it would fix #70, it would 
flash the screen when the user has flash-on-alert turned on. You can imagine 
how that would be annoying.

So, this will have to wait until we switch from SystemSound to NSSound as part 
of our burnination of deprecated API usages.

Original comment by prhgr...@gmail.com on 30 Jun 2010 at 5:47

GoogleCodeExporter commented 9 years ago

should also consider moving the audio alerts to a display style, as well as the 
implementation of allowing multiple notifications per message. (eg. audio alert 
+ prowl + smoke)

Original comment by rarich...@gmail.com on 30 Jun 2010 at 1:38

GoogleCodeExporter commented 9 years ago

Original comment by chrisf.g...@gmail.com on 29 Jul 2010 at 4:43

Added labels: Milestone-2.0

GoogleCodeExporter commented 9 years ago

How hard is that Rudy?

Original comment by chrisf.g...@gmail.com on 29 Jul 2010 at 9:58

GoogleCodeExporter commented 9 years ago

Original comment by prhgr...@gmail.com on 28 Mar 2011 at 9:31

GoogleCodeExporter commented 9 years ago

Changing the owner on these to get them out of Peter's queue.

Original comment by chrisf.g...@gmail.com on 15 Aug 2011 at 6:49

GoogleCodeExporter commented 9 years ago

Original comment by rarich...@gmail.com on 29 Aug 2011 at 4:11

Added labels: Milestone-1.3
Removed labels: Milestone-2.0

GoogleCodeExporter commented 9 years ago

fixed as of [b49474ac6c40]

Original comment by rarich...@gmail.com on 29 Aug 2011 at 10:51

Changed state: FixedInSource

GoogleCodeExporter commented 9 years ago

Original comment by ch...@growl.info on 4 Nov 2011 at 2:55

Changed state: Fixed