-
-
Notifications
You must be signed in to change notification settings - Fork 105
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.
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
Prompt text suggestions (and some for command names) #70
Comments
I have to think about it. I also would like to have short prompts.
No, I had that originally and this is confusing since commands ending in minor-mode are minor modes. Note that there is also the mouse-minor-mode-menu. If you have a better name please let me know, but
I think that name is too long. So probably I won't change the command names but the prompts could be improved. |
Good point. |
How about the name |
Or, if that still sounds like it's a minor mode, maybe |
Toggle is even longer than menu, and this is what we do - we consult the menu. Not the toggle. |
Ah! The name didn't click with me until you said "we consult the menu". Somehow I hadn't seen how that command fit the |
I close this since I am mostly satisfied by the current prompts/commands. Please let me know if you have further suggestions! |
Reading through consult a few of the prompt texts struck me as a litle off. I made a pass through the commands and have some suggestions to improve them. This is all minor stuff.
First the simple ones, just the changing the prompt text.
Imenu.el commands refer to "index item"s so I think
consult-imenu
should prompt "Go to index item: "consult--yank-read
just prompts with "Ring: " which is a little obtuse, I think "Yank text: " is clearer.consult-line
was one of the first that seemed off to me. It prompts now with "Go to line: " and at first I was thinking it was asking for a line number. "Go to line containing: " is clearer, but doesn't match with other things. I'm not completely satisfied with this suggestion, and as it stands it's consistent with other prompts, but I think it needs something because lines are so typically described by their number.These I suggest changing the command name as well.
consult-minor-mode-menu
should just beconsult-minor-mode
the wayconsult-theme
doesn't have "-menu" appended.consult-outline
currently prompts with "Go to heading: ". I think the command name should match the prompt and "heading" is more accurate. Outline mode commands say "heading" but of course they also begin with "outline". I think using both is best. If not, I think both the command name and prompt should use the same term and I prefer heading over outline since that what the list of candidates is. Soconsult-outline-heading
prompting as "Go to outline heading: "The text was updated successfully, but these errors were encountered: