[Gajim-devel] two patches

[Aleksey Rybalkin]

Hello, Yann.

> Hi Aleksey,
>
> Thanks for your patches! Let me repli in your mail.
>
> On 04/23/2012 08:03 PM, Aleksey Rybalkin wrote:
>> First is about fixing closing/reordering of stacking old-school popup
>> notifications. Rationale: I prefer to use big 'notification_timeout', so
>> popups are stacking up and when I close one which is not the bottom one
>> I
>> expect it close (not the bottom one) and not to have empty space in
>> place
>> of popup I am closing. That was old buggy behavior. Now we save index
>> for
>> each popup window and close popups based on that. Also we re-index
>> popups
>> while reordering them on screen.
>
> Perfect, I'll include that as soon as I'll test it, but it sounds perfect.
>

Thanks for applying the patch. I also discovered we now need to correct
list size checking to prevent popping out-of-bounds index. This small
patch is in attachment. I used `hg export [revision]` this time to
preserve the commit message.

>> Second is about adding new configuration option
>> 'show_popup_for_all_unread'. Rationale: I prefer to receive popup
>> notifications even after I opened the chat window after receiving the
>> first message. That is because I use tiling window manager and no panel
>> and don't have systray, so some chat window can be open on desktop 5
>> while
>> I am working on e.g. desktop 2 and I will not know that I received new
>> message in that chat.
>
> For this one, you can already do that (and much more) with the triggers
> plugin, so I don't think I'll include it ... But don't hesitate to tell
> me if triggers plugin don't fit your needs!
>

Triggers plugin worked like a charm. Thanks!

>> FYI: I based my fixes on current hg master branch and used `hg diff -c
>> [revision]` to produce .diff files. If merging from forked repository is
>> preferable I can create one on bitbucket for example. Also I suppose new
>> configuration option may have more appropriate name. Any suggestions
>> will
>> be appreciated. If there are style errors I am ready to fix them too.
>
> It's ok to send patch files, no need of a forked repository. You can
> also open tickets with the patch files attached if you want.
>
> Thanks a lot for your contribution, and don't hesitate if you want to
> help us!
>
> --
> Yann
>


-- 
Aleksey Rybalkin
rybalkin.org
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: correct_list_size_check.diff
URL: <http://www.lagaule.org/pipermail/gajim-devel/attachments/20120424/abea6d73/attachment.ksh>