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

WIP Proof of concept to show a help dialog. #37

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

Conversation

ialbors
Copy link

@ialbors ialbors commented Apr 6, 2016

'H' shows a dialog with the shortcuts

'q' dismiss the dialog

Probably the keybinding would need some reengineering (to show descriptions or group by view)

Let he know what you think.

Thanks

@fabiofalci
Copy link
Owner

It seems a good idea!

Yeah, I think the keybinding is not in a good shape currently.
I tried to enable keys configuration using a json inside ~/.sconsify/ folder however I'm not very happy with the result. I think the result is very limited and the code got very complex. So I gave up for now but I'll try to improve that at some point.

But if you manage to include these descriptions there, in the code, don't bother with the json for now, it'd be great!

https://github.com/fabiofalci/sconsify/blob/master/ui/simple/cui_actions.go#L134

'H' shows a dialog with the shortcuts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants