Hi Matthias, Well spotted. I initially decided to rely only on config.screen but somehow ended up leaving bits suggesting that notify() should take screen argument. Having said that, it's good to allow putting popups on other screens.
I went over your patch and I think better approach would be to add another (screen) dimension to notifications[] table this removes need for the loop you added to get_offset(). I pushed my version of the patch which is simpler: http://git.mercenariesguild.net/?p=awesome.git;a=blobdiff;f=lib/naughty.lua.in;h=7c4b39a0c589441bb4ec86772c7f9dcf8d13e870;hp=75184e199635f074587fab6f990d208ad2e54284;hb=af798ff71879575e719d96399e613c4141f96389;hpb=be311ece07d4112b7d9fcea4d67e8ac86faaa181 Note that this commit also includes swapping get_offset() arg positions (width before height), I had some git fun today and forgot not to do two things at the same time. Cheers, koniu -- To unsubscribe, send mail to [EMAIL PROTECTED]
