{"action":"created","issue":{"url":"https://api.github.com/repos/awesomeWM/awesome/issues/313","labels_url":"https://api.github.com/repos/awesomeWM/awesome/issues/313/labels{/name}","comments_url":"https://api.github.com/repos/awesomeWM/awesome/issues/313/comments","events_url":"https://api.github.com/repos/awesomeWM/awesome/issues/313/events","html_url":"https://github.com/awesomeWM/awesome/pull/313","id":94820764,"number":313,"title":"RFC: awful.widget.common.list_update: expose textbox to label callback","user":{"login":"blueyed","id":9766,"avatar_url":"https://avatars.githubusercontent.com/u/9766?v=3","gravatar_id":"","url":"https://api.github.com/users/blueyed","html_url":"https://github.com/blueyed","followers_url":"https://api.github.com/users/blueyed/followers","following_url":"https://api.github.com/users/blueyed/following{/other_user}","gists_url":"https://api.github.com/users/blueyed/gists{/gist_id}","starred_url":"https://api.github.com/users/blueyed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blueyed/subscriptions","organizations_url":"https://api.github.com/users/blueyed/orgs","repos_url":"https://api.github.com/users/blueyed/repos","events_url":"https://api.github.com/users/blueyed/events{/privacy}","received_events_url":"https://api.github.com/users/blueyed/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/awesomeWM/awesome/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-07-13T21:51:55Z","updated_at":"2015-07-14T19:11:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/awesomeWM/awesome/pulls/313","html_url":"https://github.com/awesomeWM/awesome/pull/313","diff_url":"https://github.com/awesomeWM/awesome/pull/313.diff","patch_url":"https://github.com/awesomeWM/awesome/pull/313.patch"},"body":"This allows the \"label\" callback to adjust the textbox itself.\r\n`tasklist_label` is changed to make use of it and supports a new style\r\nargument `font_focus`.\r\n\r\nI am not sure if that is the way to go - probably a table should be used\r\ninstead, allowing to enhance this more easily in the future.  For now, I\r\ncould also imagine to expose/pass on the imagebox for the icon.\r\nAlso, the taglist widget should get adjusted for this API then, too.\r\n\r\nTogether with #308 this allows a theme to easily configure the font to be bold for the focused tasklist entry."},"comment":{"url":"https://api.github.com/repos/awesomeWM/awesome/issues/comments/121345203","html_url":"https://github.com/awesomeWM/awesome/pull/313#issuecomment-121345203","issue_url":"https://api.github.com/repos/awesomeWM/awesome/issues/313","id":121345203,"user":{"login":"Elv13","id":340384,"avatar_url":"https://avatars.githubusercontent.com/u/340384?v=3","gravatar_id":"","url":"https://api.github.com/users/Elv13","html_url":"https://github.com/Elv13","followers_url":"https://api.github.com/users/Elv13/followers","following_url":"https://api.github.com/users/Elv13/following{/other_user}","gists_url":"https://api.github.com/users/Elv13/gists{/gist_id}","starred_url":"https://api.github.com/users/Elv13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Elv13/subscriptions","organizations_url":"https://api.github.com/users/Elv13/orgs","repos_url":"https://api.github.com/users/Elv13/repos","events_url":"https://api.github.com/users/Elv13/events{/privacy}","received_events_url":"https://api.github.com/users/Elv13/received_events","type":"User","site_admin":false},"created_at":"2015-07-14T19:11:26Z","updated_at":"2015-07-14T19:11:26Z","body":"```\r\nUsing callbacks / delegates seems to be useful, but then that's what label is about already - and is a bit confusing at first.\r\n```\r\n\r\nBut the label never reach the user, it is used internally by `tag/tasklist.lua` and that's the end of the road. It is why I think the current design of those widget is not flexible enough. Any changes, even minor, require to [monkey]patch the widget"}}