Skip to content
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

#475 fixed, Lets have a menu #476

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

#475 fixed, Lets have a menu #476

wants to merge 1 commit into from

Conversation

andreas-roehler
Copy link

No description provided.

@Fuco1
Copy link
Owner

Fuco1 commented Jun 3, 2015

Does it have to duplicate the docstrings? That's a bit unfortunate, then one would need to keep it in sync (and it bloats the file a lot!). I don't know how easymenu works but this seems suboptimal :/

@andreas-roehler
Copy link
Author

Am 03.06.2015 um 22:18 schrieb Matus Goljer:

Does it have to duplicate the docstrings?

No. Menu seems interesting for beginners mostly.
For them some info WRT to command was delivered.

Still want the docstrings deleted?

@Fuco1
Copy link
Owner

Fuco1 commented Jun 4, 2015

I get it's more convenient to see it there right away than doing C-h f but I really fear this will get out of sync very soon. Maybe there's a way to retrive docstring of a function programatically and insert that into the menu?

@andreas-roehler
Copy link
Author

Am 04.06.2015 um 19:58 schrieb Matus Goljer:

I get it's more convenient to see it there right away than doing |C-h
f| but I really fear this will get out of sync very soon. Maybe
there's a way to retrive docstring of a function programatically and
insert that into the menu?

It would be possible to refresh the menu like cheat-sheat is printed out.

That works to the extend, as naming of the commands is regular.

Pushed the code which generated basics of sp-menu, see at end of file:

https://github.com/andreas-roehler/werkstatt/blob/master/devel/ar-create-menu.el

@Fuco1
Copy link
Owner

Fuco1 commented Jun 28, 2015

Sorry I left you waiting for so long. I am now getting a bit of time, I promise I will review this by the end of next week. If you made any changes, please push it, and also rebase against the master. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

2 participants