From: SourceForge.net <no...@so...> - 2005-05-09 06:20:54
|
Patches item #1197846, was opened at 2005-05-08 21:11 Message generated for change (Settings changed) made by enix You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Enix (enix) Assigned to: Nobody/Anonymous (nobody) >Summary: [oldstatus] win32: shows unread messages in system tray Initial Comment: About two seconds after I finished my patch to add a list of messages recieved but not yet viewed to the system notification area, I noticed that someone else had written and submitted a similar patch as well. I glanced at the other patch and it appears that it displays the number of total unviewed messages. Mine goes about it a little differently and displays the screen names of everyone that has sent you a message you haven't yet viewed and the number of new messages per screen name (very much like the away message does when "queue new messages when away" is set). So, I guess choice is good. Heres my patch - it only applies against oldstatus. ---------------------------------------------------------------------- Comment By: Enix (enix) Date: 2005-05-08 21:15 Message: Logged In: YES user_id=17877 I forgot to mention the files modified. Only docklet-win32.c is modified, so, this is a relatively unintrusive patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 |
From: SourceForge.net <no...@so...> - 2005-05-16 22:52:56
|
Patches item #1197846, was opened at 2005-05-08 17:11 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Enix (enix) >Assigned to: Daniel Atallah (datallah) Summary: [oldstatus] win32: shows unread messages in system tray Initial Comment: About two seconds after I finished my patch to add a list of messages recieved but not yet viewed to the system notification area, I noticed that someone else had written and submitted a similar patch as well. I glanced at the other patch and it appears that it displays the number of total unviewed messages. Mine goes about it a little differently and displays the screen names of everyone that has sent you a message you haven't yet viewed and the number of new messages per screen name (very much like the away message does when "queue new messages when away" is set). So, I guess choice is good. Heres my patch - it only applies against oldstatus. ---------------------------------------------------------------------- Comment By: Enix (enix) Date: 2005-05-08 17:15 Message: Logged In: YES user_id=17877 I forgot to mention the files modified. Only docklet-win32.c is modified, so, this is a relatively unintrusive patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 |
From: SourceForge.net <no...@so...> - 2005-05-17 17:50:19
|
Patches item #1197846, was opened at 2005-05-08 16:11 Message generated for change (Comment added) made by datallah You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Enix (enix) Assigned to: Daniel Atallah (datallah) Summary: [oldstatus] win32: shows unread messages in system tray Initial Comment: About two seconds after I finished my patch to add a list of messages recieved but not yet viewed to the system notification area, I noticed that someone else had written and submitted a similar patch as well. I glanced at the other patch and it appears that it displays the number of total unviewed messages. Mine goes about it a little differently and displays the screen names of everyone that has sent you a message you haven't yet viewed and the number of new messages per screen name (very much like the away message does when "queue new messages when away" is set). So, I guess choice is good. Heres my patch - it only applies against oldstatus. ---------------------------------------------------------------------- >Comment By: Daniel Atallah (datallah) Date: 2005-05-17 12:50 Message: Logged In: YES user_id=325843 The functionality is kind of cool, but I can't accept this right now, basically because the state of the queuing of unread messages in cvs HEAD is in flux right now. There isn't much point applying it to the oldstatus branch if I can't also apply it to HEAD since it will go away when HEAD is finally finished. Perhaps this can be motivation for you to work on implementing the queuing in HEAD. I think that deryni was working on it (or at least involved with it), so you will probably want to consult him about the best way to proceed. ---------------------------------------------------------------------- Comment By: Enix (enix) Date: 2005-05-08 16:15 Message: Logged In: YES user_id=17877 I forgot to mention the files modified. Only docklet-win32.c is modified, so, this is a relatively unintrusive patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 |
From: SourceForge.net <no...@so...> - 2005-05-24 04:13:17
|
Patches item #1197846, was opened at 2005-05-08 17:11 Message generated for change (Comment added) made by thekingant You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed >Resolution: Rejected Priority: 5 Submitted By: Enix (enix) Assigned to: Daniel Atallah (datallah) Summary: [oldstatus] win32: shows unread messages in system tray Initial Comment: About two seconds after I finished my patch to add a list of messages recieved but not yet viewed to the system notification area, I noticed that someone else had written and submitted a similar patch as well. I glanced at the other patch and it appears that it displays the number of total unviewed messages. Mine goes about it a little differently and displays the screen names of everyone that has sent you a message you haven't yet viewed and the number of new messages per screen name (very much like the away message does when "queue new messages when away" is set). So, I guess choice is good. Heres my patch - it only applies against oldstatus. ---------------------------------------------------------------------- >Comment By: Mark Doliner (thekingant) Date: 2005-05-24 00:13 Message: Logged In: YES user_id=20979 I agree with datallah. If this patch were small I think it'd be ok, but I don't really think it's appropriate for oldstatus (the "stable" branch). And the queuing code will DEFINITELY need some work, if you have free time... ---------------------------------------------------------------------- Comment By: Daniel Atallah (datallah) Date: 2005-05-17 13:50 Message: Logged In: YES user_id=325843 The functionality is kind of cool, but I can't accept this right now, basically because the state of the queuing of unread messages in cvs HEAD is in flux right now. There isn't much point applying it to the oldstatus branch if I can't also apply it to HEAD since it will go away when HEAD is finally finished. Perhaps this can be motivation for you to work on implementing the queuing in HEAD. I think that deryni was working on it (or at least involved with it), so you will probably want to consult him about the best way to proceed. ---------------------------------------------------------------------- Comment By: Enix (enix) Date: 2005-05-08 17:15 Message: Logged In: YES user_id=17877 I forgot to mention the files modified. Only docklet-win32.c is modified, so, this is a relatively unintrusive patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1197846&group_id=235 |