{"comment":{"url":"https://api.github.com/repos/audacious-media-player/audacious-plugins/comments/20296117","html_url":"https://github.com/audacious-media-player/audacious-plugins/commit/d40cb81bcbddeba827fbe0bca37559dff3da2b9c#commitcomment-20296117","id":20296117,"user":{"login":"jlindgren90","id":1244737,"avatar_url":"https://avatars.githubusercontent.com/u/1244737?v=3","gravatar_id":"","url":"https://api.github.com/users/jlindgren90","html_url":"https://github.com/jlindgren90","followers_url":"https://api.github.com/users/jlindgren90/followers","following_url":"https://api.github.com/users/jlindgren90/following{/other_user}","gists_url":"https://api.github.com/users/jlindgren90/gists{/gist_id}","starred_url":"https://api.github.com/users/jlindgren90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlindgren90/subscriptions","organizations_url":"https://api.github.com/users/jlindgren90/orgs","repos_url":"https://api.github.com/users/jlindgren90/repos","events_url":"https://api.github.com/users/jlindgren90/events{/privacy}","received_events_url":"https://api.github.com/users/jlindgren90/received_events","type":"User","site_admin":false},"position":20,"line":167,"path":"src/qtui/menus.cc","commit_id":"d40cb81bcbddeba827fbe0bca37559dff3da2b9c","created_at":"2016-12-26T13:26:30Z","updated_at":"2016-12-26T13:26:30Z","body":"The menu doesn't call the hook.  It's the other way around, the menu connects to the hook to update the menu when the config has been updated by something else.  So the way you have it now is fine.  Can you prefix the new hooks with \"qtui\" though, i.e. \"qtui toggle menubar\"?  I'm trying to do similar in all the plugins to avoid namespace conflicts."}}
{"actor":{"display_login":"jlindgren90"}}