{"action":"created","issue":{"url":"https://api.github.com/repos/gaearon/redux-devtools/issues/41","labels_url":"https://api.github.com/repos/gaearon/redux-devtools/issues/41/labels{/name}","comments_url":"https://api.github.com/repos/gaearon/redux-devtools/issues/41/comments","events_url":"https://api.github.com/repos/gaearon/redux-devtools/issues/41/events","html_url":"https://github.com/gaearon/redux-devtools/issues/41","id":99267495,"number":41,"title":"DebugPanel should only be concerned with positioning / anchoring / hotkeys, not styling monitors.","user":{"login":"iktl","id":7263872,"avatar_url":"https://avatars.githubusercontent.com/u/7263872?v=3","gravatar_id":"","url":"https://api.github.com/users/iktl","html_url":"https://github.com/iktl","followers_url":"https://api.github.com/users/iktl/followers","following_url":"https://api.github.com/users/iktl/following{/other_user}","gists_url":"https://api.github.com/users/iktl/gists{/gist_id}","starred_url":"https://api.github.com/users/iktl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iktl/subscriptions","organizations_url":"https://api.github.com/users/iktl/orgs","repos_url":"https://api.github.com/users/iktl/repos","events_url":"https://api.github.com/users/iktl/events{/privacy}","received_events_url":"https://api.github.com/users/iktl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-08-05T18:24:45Z","updated_at":"2015-08-11T10:56:41Z","closed_at":null,"body":"Currently, `DebugPanel` passes several styles down to the Monitor in use as an artifact of the original demo put together by @gaearon. Since new Monitors are being actively developed, and I believe we should be encouraging a consistent approach to syntax like:\r\n\r\n```\r\n<DebugPanel top right bottom>\r\n   <DevTools store={store} monitor={LogMonitor} />\r\n</DebugPanel>\r\n```\r\n\r\n...where `LogMonitor` is subbed out by the developer for their preferred Monitor. \r\n\r\nTo this effect, I suggest that DebugPanel be re-tooled to be concerned only with it's positioning on screen and anchoring (it's current properties remain intact, and also hotkeys for enabling / disabling visibility. \r\n\r\nThis frees Monitors up to be singularly focused on their own appearance and behaviours within the DebugPanel, and keeps the general DX with DevTools consistent irrespective of the Monitor being used. `Diff-Monitor` is guilty of re-implementing some of these as we didn't want to write overrides to the `DebugPanel` styles coming down, so I would be the first to jump onboard with a PR if this suggestion seems valid."},"comment":{"url":"https://api.github.com/repos/gaearon/redux-devtools/issues/comments/129834868","html_url":"https://github.com/gaearon/redux-devtools/issues/41#issuecomment-129834868","issue_url":"https://api.github.com/repos/gaearon/redux-devtools/issues/41","id":129834868,"user":{"login":"gaearon","id":810438,"avatar_url":"https://avatars.githubusercontent.com/u/810438?v=3","gravatar_id":"","url":"https://api.github.com/users/gaearon","html_url":"https://github.com/gaearon","followers_url":"https://api.github.com/users/gaearon/followers","following_url":"https://api.github.com/users/gaearon/following{/other_user}","gists_url":"https://api.github.com/users/gaearon/gists{/gist_id}","starred_url":"https://api.github.com/users/gaearon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gaearon/subscriptions","organizations_url":"https://api.github.com/users/gaearon/orgs","repos_url":"https://api.github.com/users/gaearon/repos","events_url":"https://api.github.com/users/gaearon/events{/privacy}","received_events_url":"https://api.github.com/users/gaearon/received_events","type":"User","site_admin":false},"created_at":"2015-08-11T10:56:41Z","updated_at":"2015-08-11T10:56:41Z","body":"Another note is you don't _need_ to use `DebugPanel` at all. You can put DevTools right into your app."}}