Closed Bug 954601 Opened 11 years ago Closed 11 years ago

Make tab completion smarter

Categories

(Instantbird Graveyard :: Conversation, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: clokep, Assigned: aleth)

Details

Attachments

(1 file, 3 obsolete files)

*** Original post on bio 1169 at 2011-11-15 15:32:00 UTC ***

The tab completion algorithm could be smarter to take into account active participants (people who've said something recently) and also people who have recently pinged you.
Attached patch Patch (obsolete) — Splinter Review
*** Original post on bio 1169 as attmnt 1001 at 2011-11-17 21:34:00 UTC ***

When there are multiple possible completions, but only one of them is an active nick, complete with that nick, while nevertheless displaying all possible completions in a system message just in case.
Attachment #8352743 - Flags: review?(florian)
Attached patch Patch (obsolete) — Splinter Review
*** Original post on bio 1169 as attmnt 1003 at 2011-11-17 22:40:00 UTC ***

Now without system message when a whole nick is completed.

For future reference, on IRC the following alternative behaviour was also discussed: If at least one nick is active in the list of possible completions, ignore the inactive nicks.
Attachment #8352745 - Flags: review?(florian)
Comment on attachment 8352743 [details] [diff] [review]
Patch

*** Original change on bio 1169 attmnt 1001 at 2011-11-17 22:40:34 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8352743 - Attachment is obsolete: true
Attachment #8352743 - Flags: review?(florian)
Attached patch Patch (obsolete) — Splinter Review
*** Original post on bio 1169 as attmnt 1004 at 2011-11-17 22:48:00 UTC ***

Obvious mistake hopefully now absent.
Attachment #8352746 - Flags: review?(florian)
Comment on attachment 8352745 [details] [diff] [review]
Patch

*** Original change on bio 1169 attmnt 1003 at 2011-11-17 22:48:49 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8352745 - Attachment is obsolete: true
Attachment #8352745 - Flags: review?(florian)
Attached patch PatchSplinter Review
*** Original post on bio 1169 as attmnt 1005 at 2011-11-17 23:09:00 UTC ***

Obeys style guide ;)
Attachment #8352747 - Flags: review?(florian)
Comment on attachment 8352746 [details] [diff] [review]
Patch

*** Original change on bio 1169 attmnt 1004 at 2011-11-17 23:09:37 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8352746 - Attachment is obsolete: true
Attachment #8352746 - Flags: review?(florian)
Comment on attachment 8352747 [details] [diff] [review]
Patch

*** Original change on bio 1169 attmnt 1005 at 2011-11-18 00:26:29 UTC ***

Looks good!
Attachment #8352747 - Flags: review?(florian) → review+
Status: NEW → ASSIGNED
*** Original post on bio 1169 at 2011-11-18 01:12:30 UTC ***

Fixed, thanks!
https://hg.instantbird.org/instantbird/rev/b6e0a30d6b95
Assignee: nobody → aleth
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.2
*** Original post on bio 1169 at 2011-11-20 22:29:36 UTC ***

Follow-up:
https://hg.instantbird.org/instantbird/rev/e91301c49e93 - fix the completion of commands in IM conversations, as discussed on IRC.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: