New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove nesting commands from fixup/Refactor menu #765

Closed

Tracked by

abeatrix opened this issue Aug 21, 2023 · 0 comments

Closed

Tracked by

Remove nesting commands from fixup/Refactor menu #765

abeatrix opened this issue Aug 21, 2023 · 0 comments

Assignees

Labels

Contributor

abeatrix commented Aug 21, 2023 •

edited

Loading

RE: https://sourcegraph.slack.com/archives/C05AGQYD528/p1692594871104619?thread_ts=1692590131.656459&cid=C05AGQYD528

the nesting commands in fixup/refactor menu has been causing a lot of confusion:

Follow-up work from #510 (comment)

This was referenced Aug 21, 2023

Tracking issue: 8/21 - 9/4 #750

Closed

remove fixup command handling from refactor menu #766

Merged

abeatrix added a commit that referenced this issue


          remove fixup command handling from refactor menu (#766)

9753dc9

RE: #765

#510 (comment)
added the nested commands to the `Refactor Menu` with menu title `Cody`
that is similar to the `Cody Commands Menu` which has been causing a lot
of confusion.

There are follow-up works that @umpox has planned for, but before those
are completed, I proposed the following in this PR:
- remove the nested commands from the refactor menu
- rename the refactor menu from `Cody` to `Cody: Refactor Code` to avoid
confusion
- replace placeholder text for the refactor menu from `Tell Cody what to
do, or type '/' for commands` to `Enter your refactoring instruction
here...`

## Test plan

<!-- Required. See
https://docs.sourcegraph.com/dev/background-information/testing_principles.
-->

### Before


![image](https://github.com/sourcegraph/cody/assets/68532117/a34eed35-24c4-456d-b785-e1e886caf787)


### After


![image](https://github.com/sourcegraph/cody/assets/68532117/57d49521-e3b2-43dc-b71d-89ef36320a3c)


![refactormenu](https://github.com/sourcegraph/cody/assets/68532117/0e4bc151-a41a-47fe-9a33-dc3272a1c3b0)

---------

Co-authored-by: Tim Lucas <t@toolmantim.com>

jdorfman added the clients/vscode label

abeatrix self-assigned this

abeatrix closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment