Hello,

Kaushal Modi <kaushal.m...@gmail.com> writes:

> Currently if you do M-RET in a checkbox list item, the next item does not
> become a checkbox automatically. This patch fixes that.
>
> I have been using this fix locally for few months. But never got to
> submitting it as I need to yet write a test for it.
>
> Does that patch look good (apart from missing tests). Should I format it
> w.r.t to next? or master?

IMO, there is nothing to fix. Inserting a checkbox in every item is not
mandatory.

Furthermore, currently, M-RET inserts a regular item and M-S-RET inserts
it with a checkbox. With your patch, both M-RET and M-S-RET do the same
thing, which is sub-optimal.

My vote is to keep the current behaviour.

Regards,

-- 
Nicolas Goaziou

Reply via email to