{"action":"created","issue":{"url":"https://api.github.com/repos/18F/regulations-site/issues/88","repository_url":"https://api.github.com/repos/18F/regulations-site","labels_url":"https://api.github.com/repos/18F/regulations-site/issues/88/labels{/name}","comments_url":"https://api.github.com/repos/18F/regulations-site/issues/88/comments","events_url":"https://api.github.com/repos/18F/regulations-site/issues/88/events","html_url":"https://github.com/18F/regulations-site/pull/88","id":132268891,"number":88,"title":"Style refactoring - regulation nav","user":{"login":"jbarnicle","id":737400,"avatar_url":"https://avatars.githubusercontent.com/u/737400?v=3","gravatar_id":"","url":"https://api.github.com/users/jbarnicle","html_url":"https://github.com/jbarnicle","followers_url":"https://api.github.com/users/jbarnicle/followers","following_url":"https://api.github.com/users/jbarnicle/following{/other_user}","gists_url":"https://api.github.com/users/jbarnicle/gists{/gist_id}","starred_url":"https://api.github.com/users/jbarnicle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbarnicle/subscriptions","organizations_url":"https://api.github.com/users/jbarnicle/orgs","repos_url":"https://api.github.com/users/jbarnicle/repos","events_url":"https://api.github.com/users/jbarnicle/events{/privacy}","received_events_url":"https://api.github.com/users/jbarnicle/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-08T22:03:25Z","updated_at":"2016-02-09T14:39:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/18F/regulations-site/pulls/88","html_url":"https://github.com/18F/regulations-site/pull/88","diff_url":"https://github.com/18F/regulations-site/pull/88.diff","patch_url":"https://github.com/18F/regulations-site/pull/88.patch"},"body":"- Create generic mixin based font integration points for regulation nav (drawer).\r\n- Create custom-mixins.less to give the ability to overwrite or extend mixins.less\r\n\r\nGoing thru the existing css and implementing Jen's changes, it's clear that to implement any new theme we will need a granular way to modify font setting for a lot of different content.  Doing this thru variables can easily become unwieldy since the design can call for changes in size, line height, color, family, and style across any of these integration points.  This can lead to a variable explosion. Mixins give the flexibility to specify all or none of these neatly in one place.\r\n\r\nThe general approach I intended to take was apply this concept across all integration points.\r\n"},"comment":{"url":"https://api.github.com/repos/18F/regulations-site/issues/comments/181894590","html_url":"https://github.com/18F/regulations-site/pull/88#issuecomment-181894590","issue_url":"https://api.github.com/repos/18F/regulations-site/issues/88","id":181894590,"user":{"login":"cmc333333","id":326918,"avatar_url":"https://avatars.githubusercontent.com/u/326918?v=3","gravatar_id":"","url":"https://api.github.com/users/cmc333333","html_url":"https://github.com/cmc333333","followers_url":"https://api.github.com/users/cmc333333/followers","following_url":"https://api.github.com/users/cmc333333/following{/other_user}","gists_url":"https://api.github.com/users/cmc333333/gists{/gist_id}","starred_url":"https://api.github.com/users/cmc333333/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmc333333/subscriptions","organizations_url":"https://api.github.com/users/cmc333333/orgs","repos_url":"https://api.github.com/users/cmc333333/repos","events_url":"https://api.github.com/users/cmc333333/events{/privacy}","received_events_url":"https://api.github.com/users/cmc333333/received_events","type":"User","site_admin":false},"created_at":"2016-02-09T14:39:06Z","updated_at":"2016-02-09T14:39:06Z","body":"This has been reviewed by a few folks now. The expected flow:\r\n\r\n1. define core variables/mixins\r\n1. allow a hook for agencies to override\r\n1. build meaningful modules on top of those variables/mixins\r\n1. continue to allow agencies to completely override whole less files if need be"}}