[PATCH] D128621: [clangd] Do not try to use $0 as a placeholder in completion snippets

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added a comment.

My 2 cents here. We should probably try hard to keep the cursor inside braces/parentheses for those patterns.
Having

  namespace foo {
    decls
  }^ <- cursor here

hurts UX. It would be nice to get the old behavior back somehow.
I still have hopes for the VSCode discussion, maybe they can provide a workaround/willing to change their behavior. 
@kadircet also suggested trying something like `${2:decls}$0`. Could you check if that works in VSCode?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128621/new/

https://reviews.llvm.org/D128621