{"action":"created","issue":{"url":"https://api.github.com/repos/mattermost/desktop/issues/151","repository_url":"https://api.github.com/repos/mattermost/desktop","labels_url":"https://api.github.com/repos/mattermost/desktop/issues/151/labels{/name}","comments_url":"https://api.github.com/repos/mattermost/desktop/issues/151/comments","events_url":"https://api.github.com/repos/mattermost/desktop/issues/151/events","html_url":"https://github.com/mattermost/desktop/issues/151","id":156830954,"number":151,"title":"Notification design improvements for OS X","user":{"login":"kethinov","id":419567,"avatar_url":"https://avatars.githubusercontent.com/u/419567?v=3","gravatar_id":"","url":"https://api.github.com/users/kethinov","html_url":"https://github.com/kethinov","followers_url":"https://api.github.com/users/kethinov/followers","following_url":"https://api.github.com/users/kethinov/following{/other_user}","gists_url":"https://api.github.com/users/kethinov/gists{/gist_id}","starred_url":"https://api.github.com/users/kethinov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kethinov/subscriptions","organizations_url":"https://api.github.com/users/kethinov/orgs","repos_url":"https://api.github.com/users/kethinov/repos","events_url":"https://api.github.com/users/kethinov/events{/privacy}","received_events_url":"https://api.github.com/users/kethinov/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mattermost/desktop/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/mattermost/desktop/labels/os%20x","name":"os x","color":"efefef"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-05-25T19:28:48Z","updated_at":"2016-06-08T12:44:25Z","closed_at":null,"body":"The current design of the native electron-driven OS X notification could use some design improvements.\r\n\r\nHere is Slack's:\r\n\r\n![screen shot 2016-05-25 at 3 21 20 pm](https://cloud.githubusercontent.com/assets/419567/15553425/fb0dc6b2-228c-11e6-90ad-4b6aca8aaca4.png)\r\n\r\nAnd Mattermost's:\r\n\r\n![screen shot 2016-05-25 at 3 19 17 pm](https://cloud.githubusercontent.com/assets/419567/15553437/05c84abe-228d-11e6-826e-993f4c1642ec.png)\r\n\r\nSome things that might be worth doing:\r\n- One Mattermost icon instead of two.\r\n- Align the notification text to the top. (Or at least add two lines.)\r\n- Consider formatting the notification to use bold for message boilerplate and non-bold for content (similar to Slack)."},"comment":{"url":"https://api.github.com/repos/mattermost/desktop/issues/comments/224577327","html_url":"https://github.com/mattermost/desktop/issues/151#issuecomment-224577327","issue_url":"https://api.github.com/repos/mattermost/desktop/issues/151","id":224577327,"user":{"login":"yuya-oc","id":1412057,"avatar_url":"https://avatars.githubusercontent.com/u/1412057?v=3","gravatar_id":"","url":"https://api.github.com/users/yuya-oc","html_url":"https://github.com/yuya-oc","followers_url":"https://api.github.com/users/yuya-oc/followers","following_url":"https://api.github.com/users/yuya-oc/following{/other_user}","gists_url":"https://api.github.com/users/yuya-oc/gists{/gist_id}","starred_url":"https://api.github.com/users/yuya-oc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yuya-oc/subscriptions","organizations_url":"https://api.github.com/users/yuya-oc/orgs","repos_url":"https://api.github.com/users/yuya-oc/repos","events_url":"https://api.github.com/users/yuya-oc/events{/privacy}","received_events_url":"https://api.github.com/users/yuya-oc/received_events","type":"User","site_admin":false},"created_at":"2016-06-08T12:44:25Z","updated_at":"2016-06-08T12:44:25Z","body":"It seems that the commit doesn't work as I think. In my memory, Electron's `Notification.onclick` doesn't change window's focus state, so we should handle `onlick` manually in this kind of app. Then, `onNotificationClick` event should be sent on `onclick` in order to switch to the notified tab.\r\n\r\nBTW, you can use a local machine like VirtualBox or Docker to host mattermost server for testing."}}
{"actor":{"display_login":"yuya-oc"}}