smartparens icon indicating copy to clipboard operation
smartparens copied to clipboard

#475 fixed, Lets have a menu

Open andreas-roehler opened this issue 10 years ago • 5 comments

andreas-roehler avatar Jun 03 '15 20:06 andreas-roehler

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 :/

Fuco1 avatar Jun 03 '15 20:06 Fuco1

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?

andreas-roehler avatar Jun 04 '15 05:06 andreas-roehler

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?

Fuco1 avatar Jun 04 '15 17:06 Fuco1

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

andreas-roehler avatar Jun 04 '15 19:06 andreas-roehler

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!

Fuco1 avatar Jun 28 '15 20:06 Fuco1