Am 03.02.2011 01:33, André Aparício wrote:
@@ -272,7 +272,7 @@
-- @return The notification object
function notify(args)
-- gather variables together
- local preset = args.preset or config.default_preset or {}
+ local preset = args.preset
local timeout = args.timeout or preset.timeout or 5
local icon = args.icon or preset.icon
local icon_size = args.icon_size or preset.icon_size
Hi,
this change will break everything which doesn't come through dbus. Previously,
naughty.notify({ text = "foo" }) got the default preset applied, but with your
patch it wouldn't get any preset applied.
Why do you change this at all? Can't you just leave this chunk in and only apply
the other change from your patch?
Also, please send git-formatted patches. That way git will track you as the
patch's author (and instead of me, you get to write a commit message :-P).
Cheers,
Uli
--
To unsubscribe, send mail to [email protected].
