#2186 Expression Builder: Operation symbols and add field buttons cannot add at the beginning of expression

[mikieyx]

Fixes: #2186

Developer's Certificate of Origin 1.1

   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

[srikant-ch5]

LGTM.

[matthoward366]

Should the TODO be removed? Also, is there a case where cursor === 0? Wonder if this should (isNaN(cursor) || cursor === 0) ...

[mikieyx]

I'll look into it.

[mikieyx]

If I switch the condition to (isNan(cursor) || cursor === 0) , the bug is still present. I'm not to sure about the comment, I can remove if need be.

[matthoward366]

we should remove it since I don't know why it would be needed anymore.