search

fastsupply / growl

Automatically exported from code.google.com/p/growl
0 stars 0 forks source link

'Application Decides' option for priority #216

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In the current notification configuration page (as of Growl 1.2.2) for 
applications, Stay On Screen has an 'Application Decides' option, whereas 
Priority does not.
Given that an application can define both when it sends notifications, this is 
inconsistent.
Priority should also have an 'Application Decides' option (which should be the 
default).

Original issue reported on code.google.com by I.atent.dead@gmail.com on 31 May 2011 at 11:24

GoogleCodeExporter commented 9 years ago 
Priorities are going away at some point. They're only used to help pick 
different notification display styles really. How about actually just letting 
users pick notification styles and getting rid of the priorities? :)

Original comment by chrisf.g...@gmail.com on 31 May 2011 at 11:46

GoogleCodeExporter commented 9 years ago 
Chris: They're *currently* only used to pick different styles. This is not 
necessarily the case and may change, especially if/when we change to a 
rules-actions UI. For example, the user may configure a sound and/or email 
and/or SMS action for any notification whose priority is above a certain 
threshold.

Original comment by p...@growl.info on 1 Jun 2011 at 2:18

GoogleCodeExporter commented 9 years ago 
Actually there's a bit more. For the queue'ing displays, the priorities do 
matter. Higher priority items go ahead in the list. I completely forgot about 
this, and I do not remember if this is still the case or not.

Agreed on comment #2 if we got rules based.

Original comment by chrisf.g...@gmail.com on 1 Jun 2011 at 2:25

GoogleCodeExporter commented 9 years ago 
Priority is likely important for Prowl. Let's see if we can make it better.

Original comment by chrisf.g...@gmail.com on 11 Aug 2011 at 6:23

GoogleCodeExporter commented 9 years ago

Original comment by ch...@growl.info on 6 Feb 2012 at 10:37

GoogleCodeExporter commented 9 years ago 
Technically, we already had an application decides, anytime the notification's 
priority pop up hadn't been touched, while he UI showed it as normal, in 
reality it was GrowlPriorityUnset, which defaults through to the sent priority. 
 I have fixed this by removing the transformer, and adding an Application 
Decides option to the pop up (it will be the default, although it hadn't been 
shown as such), allowing re-selection of App Decides if you decide to remove 
you want to remove your priority modification.

Fixed in [fc43b3cb63f6]

Original comment by dan...@growl.info on 4 Dec 2012 at 4:09