0000121: change a bit how the auto completion adds the ) for an $iden( and put the cursor in the middle

View Issue Details [ Jump to Notes ] [ Wiki ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0000121

mSLDev

AutoComplete

public

2013-04-04 20:06

2013-04-13 02:38

Reporter

Wims

Assigned To

administrator

Priority

normal

Severity

minor

Reproducibility

N/A

Status

closed

Resolution

won't fix

Platform

OS Version

Product Version

Target Version

Fixed in Version

0.4.1.0

Summary

0000121: change a bit how the auto completion adds the ) for an $iden( and put the cursor in the middle

Description

supposing you have the following line:

alias name return |1000

where | is the cursor.

1) type $calc( and you get:

alias name return $calc(|)1000

which is not really nice as you can imagine.
I suggest it grabs the first next 'token' and puts the cursor like this:

alias name return $calc(1000|)

furthermore, the behavior could be extended like this:

after the completion by grabing the first next token, if we hold a key like shift and use the arrow keys left or right, we can move the cursor and the ) to choose up to which part of the line you want to apply the $identifier, that would be really handy:

1) alias name return |1000 / 25 * 100 % 2

2) you type $calc(, you get:

   alias name return $calc(1000|) / 25 * 100 % 2

3) you hold shift and hit the right arrow key, you get:

   alias name return $calc(1000 /|) 25 * 100 % 2

4) repeat 3) to quickly get :

  alias name return $calc(1000 / 25 * 100 % 2|)

left arrow key to move backward.
and if we are at the last token, put the cursor after the ')'.

Tags

No tags attached.

Beta mIRC version?

mIRC Version

Attached Files

Relationships

Notes
(0000082) administrator (administrator) 2013-04-13 02:26 edited on: 2013-04-13 02:38	There are too many issues: $calc(\| $foobar( ) Will now become: $calc( $foobar() ) Nested idents will cause even bigger issues: $foobar( $calc(\| ,bar ,baz) $foobar( $calc(\| ,bar) ,baz) Overall the chances that the next token is actually part of the ident is unlikely in practice. Disable the auto ) auto-complete if it annoys you.

Issue History
Date Modified	Username	Field	Change
2013-04-04 20:06	Wims	New Issue
2013-04-13 02:26	administrator	Note Added: 0000082
2013-04-13 02:38	administrator	Note Edited: 0000082	View Revisions
2013-04-13 02:38	administrator	Status	new => closed
2013-04-13 02:38	administrator	Assigned To	=> administrator
2013-04-13 02:38	administrator	Resolution	open => won't fix
2013-04-13 02:38	administrator	Fixed in Version	=> 0.4.1.0