955314 - Nick completion should add punctuation after complete nicks

Status: RESOLVED FIXED

(Instantbird Graveyard :: Conversation, enhancement)

Description

[Florian Quèze [:florian]]

*** Original post on bio 1881 at 2013-02-22 20:00:00 UTC ***

When I type jb<tab> I'm surprised that ": " isn't inserted.

Comment 1

[aleth [:aleth]]

Attachment: Patch

*** Original post on bio 1881 as attmnt 2242 at 2013-02-22 22:30:00 UTC ***

This should fix it.

Comment 2

[aleth [:aleth]]

Comment on attachment 8354006 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2242 at 2013-02-22 23:12:07 UTC ***

12:04:02 AM - flo-retina: aleth: if there's Standard8 and Standard8|mobile in the channel, and I type Standard8<tab> what's the expected behavior?

Comment 3

[aleth [:aleth]]

Attachment: Patch

*** Original post on bio 1881 as attmnt 2243 at 2013-02-22 23:41:00 UTC ***

I think this is a more consistent experience for the case of multiple possible completions, but please test!

Comment 4

[aleth [:aleth]]

Comment on attachment 8354006 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2242 at 2013-02-22 23:41:37 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 5

[aleth [:aleth]]

Attachment: Patch

*** Original post on bio 1881 as attmnt 2245 at 2013-02-23 10:46:00 UTC ***

Edge case of the edge case: If there is already punctuation, don't add any more.

Comment 6

[aleth [:aleth]]

Comment on attachment 8354007 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2243 at 2013-02-23 10:46:29 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 7

[Patrick Cloke [:clokep]]

Comment on attachment 8354009 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2245 at 2013-03-09 16:38:08 UTC ***

I just found a weird edge case with this patch applied.

In a room with me:
1. Type "clokep"
2. Press <tab> (you should now have "clokep: "
3. Press <backspace> (you should now have "clokep")
4. Press <backspace> (you still have "clokep"), I'd expect this to be "cloke".

Comment 8

[aleth [:aleth]]

Attachment: Patch

*** Original post on bio 1881 as attmnt 2262 at 2013-03-09 17:36:00 UTC ***

Handles the edge case spotted by clokep (the additional backspace key press necessary was due to undoing a tab completion of zero length).

Comment 9

[aleth [:aleth]]

Comment on attachment 8354009 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2245 at 2013-03-09 17:36:52 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 10

[aleth [:aleth]]

Attachment: Alternative patch

*** Original post on bio 1881 as attmnt 2263 at 2013-03-09 17:45:00 UTC ***

Alternative way to organize exactly the same code.

Comment 11

[aleth [:aleth]]

Comment on attachment 8354027 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2262 at 2013-03-09 18:02:21 UTC ***

First patch breaks e.g. character counts as it returns too early.

Comment 12

[Florian Quèze [:florian]]

*** Original post on bio 1881 at 2013-03-15 23:00:08 UTC ***

I tested the patch a bit.

Strange behavior:
1. Type "clokep<tab>", the result is: "clokep: "
2. Press <backspace>, the result is "clokep"
3. Press <tab> again, nothing changes (still "clokep", I would expect "clokep: ").

Comment 13

[aleth [:aleth]]

Attachment: Patch

*** Original post on bio 1881 as attmnt 2283 at 2013-03-16 15:34:00 UTC ***

Fixes that edge case. I can't see a more elegant way to do this.

Comment 14

[aleth [:aleth]]

Comment on attachment 8354028 [details] [diff] [review]
Alternative patch

*** Original change on bio 1881 attmnt 2263 at 2013-03-16 15:34:48 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 15

[Florian Quèze [:florian]]

Comment on attachment 8354048 [details] [diff] [review]
Patch

*** Original change on bio 1881 attmnt 2283 at 2013-03-21 23:20:04 UTC ***

Thanks!

Comment 16

[Patrick Cloke [:clokep]]

*** Original post on bio 1881 at 2013-03-22 10:20:58 UTC ***

http://hg.instantbird.org/instantbird/rev/c3afef072ba0

Can't wait to use this!