[Mentions] Better reply insertion intelligence

[tobyzerner]

From @tobscure on June 25, 2015 0:44

When you click "reply" on a post, it will check if there is already content in the composer; if so, it will add two blank lines before the mention. This is good, but it can be annoying if you've already put two blank lines in preparation. We should detect if there are blank lines already there or not.

What needs to be done:

• Fix logic in the insertMention function in the addPostReplyAction initializer.

Copied from original issue: flarum/mentions#5

[franzliedke]

Fixed in flarum/mentions@e825d1a.

I changed it so that at most two newlines are inserted, depending on the number of preceding newlines. If more than two newlines precede the cursor, they are left intact.

[franzliedke]

@tobscure Oh, should we take similar caution for any already existing content following the cursor (i.e. inserting newlines after the quote), too?

[tobyzerner]

I suppose so!