mirror of
https://github.com/zed-industries/zed.git
synced 2026-07-30 10:07:43 +00:00
Closes #61312 In `parse_choices`, the escape branch only appended the escaped character to `current_choice` when parsing the first choice, because the push was placed inside the `!found_default_choice` guard. That guard is only meant for `text` (the default value inserted into the buffer) — the plain-text branch below handles it that way. As a result, `${1|a,b\,c|}` offered the choices `a` and `bc` instead of `a` and `b,c`, and the same applied to any `\,`, `\|`, or `\\` in choices after the first. Moved `current_choice.push(c)` out of the guard and added a regression test covering escaped comma, pipe, and backslash in non-first choices. Escapes in the first choice were already covered by the existing test and are unchanged. Before / after (snippet body `"${1|a,b\\,c|}"`): the choice dropdown showed `a` / `bc`, now shows `a` / `b,c`. Release Notes: - Fixed escaped characters (`\,`, `\|`, `\\`) being dropped from snippet choice options after the first one |
||
|---|---|---|
| .. | ||
| src | ||
| Cargo.toml | ||
| LICENSE-GPL | ||