ivy.el (ivy-completion-in-region): Allow sorting

[basil-conto]

(ivy-sort-functions-alist): Default ivy-completion-in-region to no sorting.

[abo-abo]

What's the reason for this change?

[basil-conto]

What's the reason for this change?

As with any addition of :sort t, it is to allow users to customise the sort function for those in-buffer completion functions which end up using ivy-completion-in-region, such as elisp-completion-at-point.

Before adding :sort t:

After adding :sort t and (map-put ivy-sort-functions-alist #'ivy-completion-in-region #'string-lessp):

Is there any reason why ivy-completing-read should have :sort t and not ivy-completion-in-region?

[basil-conto]

(Note that, in my example, adding an entry for elisp-completion-at-point in ivy-sort-functions-alist does not work because (ivy-state-caller ivy-last) is ivy-completion-in-region.)

[abo-abo]

Thanks.

[dieggsy]

Thanks for this @basil-conto