Menu
▾
▴
pidgin-patches — Updates to the Gaim patches tracker
You can subscribe to this list here.
| 2004 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
(435) |
Dec
(252) |
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 2005 |
Jan
(177) |
Feb
(157) |
Mar
(187) |
Apr
(168) |
May
(127) |
Jun
(291) |
Jul
(38) |
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
|
From: SourceForge.net <no...@so...> - 2005-04-07 05:18:40
|
Patches item #1178273, was opened at 2005-04-07 00:18 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178273&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: Autocomplete Accounts Initial Comment: When you autocomplete a screenname in a dialog box that also includes an account selector, it should also autocomplete the account. This patch implements that. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178273&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-07 03:41:50
|
Patches item #1178248, was opened at 2005-04-06 22:41 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178248&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: Matt Amato (matt_amato) Assigned to: Nobody/Anonymous (nobody) Summary: [oldstatus] Signals for Add/Remove Buddy Initial Comment: This is a simple patch that registers two new signals. One for when a buddy is added to the buddy list and another for when he/she is removed. The added signal is emitted after the buddy is fully added and the remove signal is emitted after the buddy is removed but just before the memory associated with the buddy is freed. I also updated signals-test.c to register and test these signals. This is also my first patch for any OSS project, so go easy on me. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178248&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-07 03:20:13
|
Patches item #1176125, was opened at 2005-04-03 21:33 Message generated for change (Comment added) made by rlaager You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Sean Egan (seanegan) Summary: Text Replacement Plugin Fixes Initial Comment: This patch is based off xkahn's work available at: http://sf.net/support/tracker.php?aid=300235 Most of the changes are his. I just changed his formatting to match the rest of Gaim more closely. Also, I made a couple coding style changes. The two biggest changes I made were: 1) I added some hash stuff to load_config. This removes duplicates. 2) I imported my list of text replacement strings as the default list in addition to xkahn's list. My list consisted primarily of the autocorrect entries from my copy of OpenOffice.org Writer. There were a few things I've added by hand. (For those familiar with my comments in #gaim, this list does NOT include anything like a pwn -> own replacement. If people want to talk like that, I'm not going to stop them. This list should be useful to anyone.) I also fixed a memory leak in load_config. A GtkListStore stores its own copy of strings, so there is no need to g_strdup them when appending to the GtkListStore. When I get a chance, I need to make it give an error if you try to set a duplicate "You type" word in the preferences. Otherwise, this patch should take care of everything. I would recommend considering this patch over xkahn's because it's a direct superset of his patches. However, I want to be very clear that most of the changes in this patch are his and he deserves the credit for them. ---------------------------------------------------------------------- >Comment By: Richard Laager (rlaager) Date: 2005-04-06 22:20 Message: Logged In: YES user_id=156487 I fixed the problem that xkahn pointed out here. I had discovered that independently; I only saw his posting now. Yes, xkahn, it was fairly simple to fix. I also fixed the last item from my TODO list. It now presents an error if you try to add a duplicate "BAD" word. Finally, I've merged in some more autocorrections from word processors. As I noted in a comment, the ones with spaces don't currently work, but I'm leaving them in case someone modifies the plugin in the future to handle that case. I think it's good enough for me, so I'm not going to put in the work to handle multi-word corrections as that opens a giant can of worms. Oh, this patch also adds Benjamin Kahn to the COPYRIGHT file. This patch should be directly committable now. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-04-03 22:19 Message: Logged In: YES user_id=7632 So there is a major problem with my patch which your patch hasn't fixed yet. I included an "undo" feature which allows users to cancel changes to auto-correct if they hit backspace right after the change is made. The problem is that the undo feature is a little too enthusiastic. It will replace a word if you hit space even after you have moved the cursor. (Possibly only after moving the cursor with the pointer, although I no longer remember.) Also, if you hit backspace to get the word uncorrected, and then hit space again to advance, the word is auto-corrected once more. Not very friendly. This shouldn't be too hard to fix, but I haven't bothered since the patch shows no signs of being incorporated. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-07 01:40:50
|
Patches item #1178217, was opened at 2005-04-06 20:40 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178217&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: [HEAD] Signal Short Circuiting Initial Comment: The following log snippets should explain it: (20:24:00) rlaager: Regarding the signal handling conversation the other day... I've written a patch to stop calling signal handlers and return as soon as we find one signal handler that returns TRUE to indicate that it's handled the signal. Is this the right approach? (20:24:22) Ethan Blanton (Paco-Paco): the trouble is that it's documented to behave exactly the way it does (20:24:31) Ethan Blanton (Paco-Paco): so changing it is notbackwards compatible (20:24:31) rlaager: I'm talking for HEAD. (20:24:41) Ethan Blanton (Paco-Paco): oh, I think that's a good approach, yes (20:24:53) rlaager: The way I've described is how I *expected* it to work, having not read the documentation. (20:25:09) Ethan Blanton (Paco-Paco): I'm convinced (20:27:04) Stu Tomlinson (nosnilmot): rlaager: this, I assume, breaks the generic-ness of signals, by assuming that any that return values return booleans? (20:27:26) Ethan Blanton (Paco-Paco): please break it (20:27:33) Ethan Blanton (Paco-Paco): we already have out-parameters (20:27:42) rlaager: nosnilmot: from what I can see, the return type is handled as a (void *)... so I'm checking that ret_value != NULL (20:27:57) rlaager: nosnilmot: that's the correct way to do it, right? ... (20:29:01) Ethan Blanton (Paco-Paco): allowing a meaningful return value is an over-engineering (20:30:07) rlaager: even after this patch, you should be able to return meaningful return values (20:30:15) rlaager: it'll just short-circuit on the first handler that does ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178217&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-07 01:10:14
|
Patches item #1178208, was opened at 2005-04-06 20:10 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178208&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: gaim_cmd_do_command Documentation Fix Initial Comment: You can't pass NULL as errormsg as the documentation implies. If you do, it segfaults. Perhaps that should be allowable, but as the code stands, it's not. The documentation should be updated in HEAD and oldstatus to reflect this. Attached is a patch to that effect. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1178208&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 12:37:42
|
Patches item #1176798, was opened at 2005-04-05 00:37 Message generated for change (Comment added) made by nilbus0 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: Nilbus (nilbus0) Assigned to: Mark Doliner (thekingant) Summary: Timestamp on chat invitations Initial Comment: It's useful when you come back to your computer and see a chat invite to know whether it's old or recent. I wrote this simple patch to include the date and time on the chat invitation. I used %c for locale formatting. Ed Anderson ---------------------------------------------------------------------- >Comment By: Nilbus (nilbus0) Date: 2005-04-05 08:37 Message: Logged In: YES user_id=1050753 After playing with it a little, I like the strftime format "%I:%M%P on %b %d" better than "%c". Please use that format instead, if you agree. ---------------------------------------------------------------------- Comment By: Luke Schierer (lschiere) Date: 2005-04-05 07:25 Message: Logged In: YES user_id=28833 Mark, can you look this over? ---------------------------------------------------------------------- Comment By: Nilbus (nilbus0) Date: 2005-04-05 00:47 Message: Logged In: YES user_id=1050753 This was patched against the oldstatus tag of the CVS ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 11:25:59
|
Patches item #1176798, was opened at 2005-04-05 00:37 Message generated for change (Comment added) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: Nilbus (nilbus0) >Assigned to: Mark Doliner (thekingant) Summary: Timestamp on chat invitations Initial Comment: It's useful when you come back to your computer and see a chat invite to know whether it's old or recent. I wrote this simple patch to include the date and time on the chat invitation. I used %c for locale formatting. Ed Anderson ---------------------------------------------------------------------- >Comment By: Luke Schierer (lschiere) Date: 2005-04-05 07:25 Message: Logged In: YES user_id=28833 Mark, can you look this over? ---------------------------------------------------------------------- Comment By: Nilbus (nilbus0) Date: 2005-04-05 00:47 Message: Logged In: YES user_id=1050753 This was patched against the oldstatus tag of the CVS ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 05:02:13
|
Patches item #1176803, was opened at 2005-04-05 00:50 Message generated for change (Comment added) made by thekingant You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176803&group_id=235 Category: None Group: None >Status: Closed >Resolution: Accepted Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: [HEAD] Yahoo Debug Message (one-liner) Initial Comment: This patch puts this debugging output where it belongs. ---------------------------------------------------------------------- >Comment By: Mark Doliner (thekingant) Date: 2005-04-05 01:02 Message: Logged In: YES user_id=20979 Good catch, thanks ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176803&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 04:50:28
|
Patches item #1176803, was opened at 2005-04-04 23:50 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176803&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: [HEAD] Yahoo Debug Message (one-liner) Initial Comment: This patch puts this debugging output where it belongs. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176803&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 04:47:11
|
Patches item #1176798, was opened at 2005-04-05 00:37 Message generated for change (Comment added) made by nilbus0 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: Nilbus (nilbus0) Assigned to: Nobody/Anonymous (nobody) Summary: Timestamp on chat invitations Initial Comment: It's useful when you come back to your computer and see a chat invite to know whether it's old or recent. I wrote this simple patch to include the date and time on the chat invitation. I used %c for locale formatting. Ed Anderson ---------------------------------------------------------------------- >Comment By: Nilbus (nilbus0) Date: 2005-04-05 00:47 Message: Logged In: YES user_id=1050753 This was patched against the oldstatus tag of the CVS ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-05 04:37:20
|
Patches item #1176798, was opened at 2005-04-05 00:37 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: Nilbus (nilbus0) Assigned to: Nobody/Anonymous (nobody) Summary: Timestamp on chat invitations Initial Comment: It's useful when you come back to your computer and see a chat invite to know whether it's old or recent. I wrote this simple patch to include the date and time on the chat invitation. I used %c for locale formatting. Ed Anderson ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176798&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 12:26:25
|
Patches item #1176212, was opened at 2005-04-04 04:31 Message generated for change (Comment added) made by thekingant You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 Category: None Group: None Status: Closed Resolution: Accepted Priority: 5 Submitted By: Gary Kramlich (amc_grim) >Assigned to: Luke Schierer (lschiere) Summary: Link dnd fixes and then some... Initial Comment: Em was in #gaim with a crash on dnd urls, so I walked 'em through avoiding the crash. Then we discovered it was adding it twice, and realized that the second part was the data for the A tag. And then I found a memory leak. So I tweaked the patch to use the label that we're given. In other words, crash fix, memory leak fix, use the label for a url if we're given it. Things to note, selecting a big block of text with multiple links in it has issues as does selecting multiple links. I'm not sure how to go about fixing that, I leave that to someone who knows what the hell they're doing with dnd, but i got the same results with firefox and galeon. Selecting multiple links will use everything that was selected as the label, and the url for whatever part you grabbed. When selecing text with multiple links in it, the output depends on how you selected it and which part you grabbed to drag. ---------------------------------------------------------------------- >Comment By: Mark Doliner (thekingant) Date: 2005-04-04 08:26 Message: Logged In: YES user_id=20979 Should this be applied to HEAD, also? ---------------------------------------------------------------------- Comment By: Luke Schierer (lschiere) Date: 2005-04-04 07:20 Message: Logged In: YES user_id=28833 you rock. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 11:20:28
|
Patches item #1176212, was opened at 2005-04-04 04:31 Message generated for change (Comment added) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 Category: None Group: None >Status: Closed >Resolution: Accepted Priority: 5 Submitted By: Gary Kramlich (amc_grim) Assigned to: Nobody/Anonymous (nobody) Summary: Link dnd fixes and then some... Initial Comment: Em was in #gaim with a crash on dnd urls, so I walked 'em through avoiding the crash. Then we discovered it was adding it twice, and realized that the second part was the data for the A tag. And then I found a memory leak. So I tweaked the patch to use the label that we're given. In other words, crash fix, memory leak fix, use the label for a url if we're given it. Things to note, selecting a big block of text with multiple links in it has issues as does selecting multiple links. I'm not sure how to go about fixing that, I leave that to someone who knows what the hell they're doing with dnd, but i got the same results with firefox and galeon. Selecting multiple links will use everything that was selected as the label, and the url for whatever part you grabbed. When selecing text with multiple links in it, the output depends on how you selected it and which part you grabbed to drag. ---------------------------------------------------------------------- >Comment By: Luke Schierer (lschiere) Date: 2005-04-04 07:20 Message: Logged In: YES user_id=28833 you rock. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 08:31:30
|
Patches item #1176212, was opened at 2005-04-04 03:31 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Gary Kramlich (amc_grim) Assigned to: Nobody/Anonymous (nobody) Summary: Link dnd fixes and then some... Initial Comment: Em was in #gaim with a crash on dnd urls, so I walked 'em through avoiding the crash. Then we discovered it was adding it twice, and realized that the second part was the data for the A tag. And then I found a memory leak. So I tweaked the patch to use the label that we're given. In other words, crash fix, memory leak fix, use the label for a url if we're given it. Things to note, selecting a big block of text with multiple links in it has issues as does selecting multiple links. I'm not sure how to go about fixing that, I leave that to someone who knows what the hell they're doing with dnd, but i got the same results with firefox and galeon. Selecting multiple links will use everything that was selected as the label, and the url for whatever part you grabbed. When selecing text with multiple links in it, the output depends on how you selected it and which part you grabbed to drag. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176212&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 03:19:53
|
Patches item #1176125, was opened at 2005-04-04 02:33 Message generated for change (Comment added) made by xkahn You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Sean Egan (seanegan) Summary: Text Replacement Plugin Fixes Initial Comment: This patch is based off xkahn's work available at: http://sf.net/support/tracker.php?aid=300235 Most of the changes are his. I just changed his formatting to match the rest of Gaim more closely. Also, I made a couple coding style changes. The two biggest changes I made were: 1) I added some hash stuff to load_config. This removes duplicates. 2) I imported my list of text replacement strings as the default list in addition to xkahn's list. My list consisted primarily of the autocorrect entries from my copy of OpenOffice.org Writer. There were a few things I've added by hand. (For those familiar with my comments in #gaim, this list does NOT include anything like a pwn -> own replacement. If people want to talk like that, I'm not going to stop them. This list should be useful to anyone.) I also fixed a memory leak in load_config. A GtkListStore stores its own copy of strings, so there is no need to g_strdup them when appending to the GtkListStore. When I get a chance, I need to make it give an error if you try to set a duplicate "You type" word in the preferences. Otherwise, this patch should take care of everything. I would recommend considering this patch over xkahn's because it's a direct superset of his patches. However, I want to be very clear that most of the changes in this patch are his and he deserves the credit for them. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-04-04 03:19 Message: Logged In: YES user_id=7632 So there is a major problem with my patch which your patch hasn't fixed yet. I included an "undo" feature which allows users to cancel changes to auto-correct if they hit backspace right after the change is made. The problem is that the undo feature is a little too enthusiastic. It will replace a word if you hit space even after you have moved the cursor. (Possibly only after moving the cursor with the pointer, although I no longer remember.) Also, if you hit backspace to get the word uncorrected, and then hit space again to advance, the word is auto-corrected once more. Not very friendly. This shouldn't be too hard to fix, but I haven't bothered since the patch shows no signs of being incorporated. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 02:37:51
|
Patches item #1166073, was opened at 2005-03-18 10:38 Message generated for change (Comment added) made by rlaager You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1166073&group_id=235 Category: Plugins Group: None Status: Open Resolution: None Priority: 5 Submitted By: Benjamin Kahn (xkahn) Assigned to: Sean Egan (seanegan) Summary: Make the autocorrect plugin work Initial Comment: Autocorrect in gaim... um... sucks. The problem is that the correction only happens AFTER you send a message. This means that you don't actually know what message you are sending until it is sent. Horrible. My new patch works AS YOU TYPE with the following rules: * You must be typing, not pasting in text. If you paste in teh and there is a rule to correct teh to the, it won't be corrected. This is because it would be impossible to go back and check all the text. * Check whole words ONLY. Partial words don't count. * If the dictionary word has capital letters in it, match only words with that capitalization. (FPT -> FTP doesn't match ftp.) * If the dictionary word doesn't have capital letters, preserve the capitalization the user used. (The code assumes three possible capitalization schemes: ALLCAPS, lowercase, and Proper.) Unless the solution has capital letters. In that case, force the capitalization used in the solution. * A word may have quote characters in it and still be a word. * If the user immediately hits the backspace key after a correction, undo the correction. ---------------------------------------------------------------------- Comment By: Richard Laager (rlaager) Date: 2005-04-03 21:37 Message: Logged In: YES user_id=156487 xkahn, this is beautiful work. It's exactly what I was looking to do myself. I'm glad you wrote it and saved me the work! I've cleaned up this patch a bit to make it conform to the coding style in use in Gaim. I made a couple other little fixes and added a couple items off my TODO list. This patch (a strict* superset of the changes applied by these two patches) is available at: http://sf.net/support/tracker.php?aid=1176125 * Except for the fact that there were a couple typos in the correction list in gaim-0.81-defaults.patch. Yes, I mean typos in the replacement string, not the string to be replaced. I found it slightly ironic. ---------------------------------------------------------------------- Comment By: Richard Laager (rlaager) Date: 2005-04-03 14:58 Message: Logged In: YES user_id=156487 Hmm, it seems you beat me to all the things I was going to do with the text replacement plugin. I haven't yet looked at this patch, but from the description it sounds nice. I have a large list of autocorrections I've been testing. Feel free to e-mail me and I'll send it your way. Otherwise, I'll submit a patch with them if/when this is accepted. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-03-18 10:49 Message: Logged In: YES user_id=7632 Umm... Regarding the first point: * You must be typing, not pasting in text. If you paste in teh and there is a rule to correct teh to the, it won't be corrected. This is because it would be impossible to go back and check all the text. It is possible for the code to do this, however it purposely does NOT. This is because because pasted text is normally fairly long and may have multiple subtle corrections. This makes it difficult for users to understand what is happening and spot errors. Also, pasted text is often either wrong on purpose (and then he said: "hey d00d!"), or correct already. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-03-18 10:41 Message: Logged In: YES user_id=7632 Oh! And we'll need some good text replacements. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1166073&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 02:36:21
|
Patches item #1176125, was opened at 2005-04-03 22:33 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) >Assigned to: Sean Egan (seanegan) Summary: Text Replacement Plugin Fixes Initial Comment: This patch is based off xkahn's work available at: http://sf.net/support/tracker.php?aid=300235 Most of the changes are his. I just changed his formatting to match the rest of Gaim more closely. Also, I made a couple coding style changes. The two biggest changes I made were: 1) I added some hash stuff to load_config. This removes duplicates. 2) I imported my list of text replacement strings as the default list in addition to xkahn's list. My list consisted primarily of the autocorrect entries from my copy of OpenOffice.org Writer. There were a few things I've added by hand. (For those familiar with my comments in #gaim, this list does NOT include anything like a pwn -> own replacement. If people want to talk like that, I'm not going to stop them. This list should be useful to anyone.) I also fixed a memory leak in load_config. A GtkListStore stores its own copy of strings, so there is no need to g_strdup them when appending to the GtkListStore. When I get a chance, I need to make it give an error if you try to set a duplicate "You type" word in the preferences. Otherwise, this patch should take care of everything. I would recommend considering this patch over xkahn's because it's a direct superset of his patches. However, I want to be very clear that most of the changes in this patch are his and he deserves the credit for them. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 02:33:30
|
Patches item #1176125, was opened at 2005-04-03 21:33 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: Text Replacement Plugin Fixes Initial Comment: This patch is based off xkahn's work available at: http://sf.net/support/tracker.php?aid=300235 Most of the changes are his. I just changed his formatting to match the rest of Gaim more closely. Also, I made a couple coding style changes. The two biggest changes I made were: 1) I added some hash stuff to load_config. This removes duplicates. 2) I imported my list of text replacement strings as the default list in addition to xkahn's list. My list consisted primarily of the autocorrect entries from my copy of OpenOffice.org Writer. There were a few things I've added by hand. (For those familiar with my comments in #gaim, this list does NOT include anything like a pwn -> own replacement. If people want to talk like that, I'm not going to stop them. This list should be useful to anyone.) I also fixed a memory leak in load_config. A GtkListStore stores its own copy of strings, so there is no need to g_strdup them when appending to the GtkListStore. When I get a chance, I need to make it give an error if you try to set a duplicate "You type" word in the preferences. Otherwise, this patch should take care of everything. I would recommend considering this patch over xkahn's because it's a direct superset of his patches. However, I want to be very clear that most of the changes in this patch are his and he deserves the credit for them. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1176125&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-04 02:18:47
|
Patches item #1175960, was opened at 2005-04-03 16:01 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175960&group_id=235 Category: None Group: None >Status: Closed >Resolution: Accepted Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: HEAD Idle in Tooltips Initial Comment: It looks silly to have "Idle" in a tooltip by itself only to show "Status: Idle" later on. This patch corrects this. NOTE: This patch doesn't make sense for oldstatus. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175960&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-03 20:01:33
|
Patches item #1175960, was opened at 2005-04-03 15:01 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175960&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Richard Laager (rlaager) Assigned to: Nobody/Anonymous (nobody) Summary: HEAD Idle in Tooltips Initial Comment: It looks silly to have "Idle" in a tooltip by itself only to show "Status: Idle" later on. This patch corrects this. NOTE: This patch doesn't make sense for oldstatus. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175960&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-03 19:58:52
|
Patches item #1166073, was opened at 2005-03-18 10:38 Message generated for change (Comment added) made by rlaager You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1166073&group_id=235 Category: Plugins Group: None Status: Open Resolution: None Priority: 5 Submitted By: Benjamin Kahn (xkahn) Assigned to: Sean Egan (seanegan) Summary: Make the autocorrect plugin work Initial Comment: Autocorrect in gaim... um... sucks. The problem is that the correction only happens AFTER you send a message. This means that you don't actually know what message you are sending until it is sent. Horrible. My new patch works AS YOU TYPE with the following rules: * You must be typing, not pasting in text. If you paste in teh and there is a rule to correct teh to the, it won't be corrected. This is because it would be impossible to go back and check all the text. * Check whole words ONLY. Partial words don't count. * If the dictionary word has capital letters in it, match only words with that capitalization. (FPT -> FTP doesn't match ftp.) * If the dictionary word doesn't have capital letters, preserve the capitalization the user used. (The code assumes three possible capitalization schemes: ALLCAPS, lowercase, and Proper.) Unless the solution has capital letters. In that case, force the capitalization used in the solution. * A word may have quote characters in it and still be a word. * If the user immediately hits the backspace key after a correction, undo the correction. ---------------------------------------------------------------------- Comment By: Richard Laager (rlaager) Date: 2005-04-03 14:58 Message: Logged In: YES user_id=156487 Hmm, it seems you beat me to all the things I was going to do with the text replacement plugin. I haven't yet looked at this patch, but from the description it sounds nice. I have a large list of autocorrections I've been testing. Feel free to e-mail me and I'll send it your way. Otherwise, I'll submit a patch with them if/when this is accepted. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-03-18 10:49 Message: Logged In: YES user_id=7632 Umm... Regarding the first point: * You must be typing, not pasting in text. If you paste in teh and there is a rule to correct teh to the, it won't be corrected. This is because it would be impossible to go back and check all the text. It is possible for the code to do this, however it purposely does NOT. This is because because pasted text is normally fairly long and may have multiple subtle corrections. This makes it difficult for users to understand what is happening and spot errors. Also, pasted text is often either wrong on purpose (and then he said: "hey d00d!"), or correct already. ---------------------------------------------------------------------- Comment By: Benjamin Kahn (xkahn) Date: 2005-03-18 10:41 Message: Logged In: YES user_id=7632 Oh! And we'll need some good text replacements. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1166073&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-02 21:11:56
|
Patches item #1175520, was opened at 2005-04-02 15:57 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175520&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: dennisne (dennisne) >Assigned to: Sean Egan (seanegan) Summary: autoaccept file transfers Initial Comment: This patch (against gaim 2.0.0cvs-2005-04-02) adds an option to autoaccept file transfers, in Preferences > Conversations tab > File Transfers. gaim 2.0.0 has preferences listed in tabs, instead of in a list like the 1.x series ... and I didn't want to make the window wider =| ... although i did have doubts about bunching it in with the Conversation preferences. The gaim preference for this feature is: /gaim/gtk/conversations/autoaccept_transfers it is disabled by default. In Preferences, the download directory can also be specified. For this i added gaim_request_folder(), which adds support for gtk's folder-selection dialog. The default download folder, should one not be specified, is the user's home directory. All in all, autoaccepting of file transfers should be a part of gaim. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175520&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-02 20:57:54
|
Patches item #1175520, was opened at 2005-04-02 15:57 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175520&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: dennisne (dennisne) Assigned to: Nobody/Anonymous (nobody) Summary: autoaccept file transfers Initial Comment: This patch (against gaim 2.0.0cvs-2005-04-02) adds an option to autoaccept file transfers, in Preferences > Conversations tab > File Transfers. gaim 2.0.0 has preferences listed in tabs, instead of in a list like the 1.x series ... and I didn't want to make the window wider =| ... although i did have doubts about bunching it in with the Conversation preferences. The gaim preference for this feature is: /gaim/gtk/conversations/autoaccept_transfers it is disabled by default. In Preferences, the download directory can also be specified. For this i added gaim_request_folder(), which adds support for gtk's folder-selection dialog. The default download folder, should one not be specified, is the user's home directory. All in all, autoaccepting of file transfers should be a part of gaim. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1175520&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-04-02 16:41:23
|
Patches item #1149564, was opened at 2005-02-22 22:53 Message generated for change (Comment added) made by nosnilmot You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1149564&group_id=235 Category: None Group: None >Status: Closed >Resolution: Accepted Priority: 5 Submitted By: Felipe Contreras (revo) Assigned to: Stu Tomlinson (nosnilmot) Summary: [MSN] Cleanups and fixes Initial Comment: I know it fixes a multiple conversation close notification but it probably fixes more stuff. I don't know exactly what are the diferences with what Stu did. ---------------------------------------------------------------------- >Comment By: Stu Tomlinson (nosnilmot) Date: 2005-04-02 11:41 Message: Logged In: YES user_id=309779 Finally, I got around to this, sorry for the delay. Thanks! ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1149564&group_id=235 |
|
From: SourceForge.net <no...@so...> - 2005-03-29 04:05:27
|
Patches item #1172190, was opened at 2005-03-28 18:51 Message generated for change (Settings changed) made by lschiere You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1172190&group_id=235 Category: newfeature Group: None Status: Open Resolution: None Priority: 5 Submitted By: fred hampton (fredhampton) >Assigned to: Tim Ringenbach (marv_sf) Summary: Secure Yahoo mail login Initial Comment: I suggest changing the default yahoo mail URL to be that of the secure login, rather than that of the insecure, http version. This is as simple as changing the "http" in the current URL to "https". IMO, the added security is worth the one extra byte of storage space ;) Here are the exact deatils: In file "gaim-1.1.4/src/protocols/yahoo/yahoo.h" line 33 change to: #define YAHOO_MAIL_URL "https://mail.yahoo.com/" ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1172190&group_id=235 |
3 messages has been excluded from this view by a project administrator.
