{"action":"created","issue":{"url":"https://api.github.com/repos/tidepool-org/blip/issues/185","labels_url":"https://api.github.com/repos/tidepool-org/blip/issues/185/labels{/name}","comments_url":"https://api.github.com/repos/tidepool-org/blip/issues/185/comments","events_url":"https://api.github.com/repos/tidepool-org/blip/issues/185/events","html_url":"https://github.com/tidepool-org/blip/pull/185","id":54124635,"number":185,"title":"two-tone color scheme for trends view days of the week","user":{"login":"jebeck","id":1588547,"avatar_url":"https://avatars.githubusercontent.com/u/1588547?v=3","gravatar_id":"","url":"https://api.github.com/users/jebeck","html_url":"https://github.com/jebeck","followers_url":"https://api.github.com/users/jebeck/followers","following_url":"https://api.github.com/users/jebeck/following{/other_user}","gists_url":"https://api.github.com/users/jebeck/gists{/gist_id}","starred_url":"https://api.github.com/users/jebeck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jebeck/subscriptions","organizations_url":"https://api.github.com/users/jebeck/orgs","repos_url":"https://api.github.com/users/jebeck/repos","events_url":"https://api.github.com/users/jebeck/events{/privacy}","received_events_url":"https://api.github.com/users/jebeck/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-01-12T23:50:43Z","updated_at":"2015-01-13T00:22:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tidepool-org/blip/pulls/185","html_url":"https://github.com/tidepool-org/blip/pull/185","diff_url":"https://github.com/tidepool-org/blip/pull/185.diff","patch_url":"https://github.com/tidepool-org/blip/pull/185.patch"},"body":"In conjunction with [tideline PR #197](https://github.com/tidepool-org/tideline/pull/197). That should be merged and tagged first, then this PR updated.\r\n\r\nAt present, the sub-navigation for selecting and deselecting days of the week - in particular, the dark text for weekday abbrevs on the darker turquoise - does not meet our accessibility requirements for text contrast:\r\n![screenshot 2015-01-12 15 33 50](https://cloud.githubusercontent.com/assets/1588547/5713543/3f3767b6-9a72-11e4-9a57-2dbc38f7f6d6.png)\r\n\r\nI attempted a quick solution (make the day of week text white for weekdays), but I don't think it works very well:\r\n![screenshot 2015-01-12 15 38 20](https://cloud.githubusercontent.com/assets/1588547/5713547/4ba041ee-9a72-11e4-8d88-6f27136ee216.png)\r\n(White text on the weekend color also fails our accessibility requirements.)\r\n\r\nFurther input required to move forward... Ping @skrugman @brandonarbiter \r\n"},"comment":{"url":"https://api.github.com/repos/tidepool-org/blip/issues/comments/69673914","html_url":"https://github.com/tidepool-org/blip/pull/185#issuecomment-69673914","issue_url":"https://api.github.com/repos/tidepool-org/blip/issues/185","id":69673914,"user":{"login":"skrugman","id":5529057,"avatar_url":"https://avatars.githubusercontent.com/u/5529057?v=3","gravatar_id":"","url":"https://api.github.com/users/skrugman","html_url":"https://github.com/skrugman","followers_url":"https://api.github.com/users/skrugman/followers","following_url":"https://api.github.com/users/skrugman/following{/other_user}","gists_url":"https://api.github.com/users/skrugman/gists{/gist_id}","starred_url":"https://api.github.com/users/skrugman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/skrugman/subscriptions","organizations_url":"https://api.github.com/users/skrugman/orgs","repos_url":"https://api.github.com/users/skrugman/repos","events_url":"https://api.github.com/users/skrugman/events{/privacy}","received_events_url":"https://api.github.com/users/skrugman/received_events","type":"User","site_admin":false},"created_at":"2015-01-13T00:22:22Z","updated_at":"2015-01-13T00:22:22Z","body":"Sorry yeah. It does pass. Use that if you haven't already pushed it.\r\n\r\n*Sara Krugman *\r\nLead designer\r\n\r\n*Tidepool *An open source, not-for-profit effort to build an open data\r\nplatform and better applications that reduce the burden of Type 1 Diabetes\r\nand accelerate the commercialization of closed-loop systems.\r\n\r\n*Phone :* 617 549 1483\r\n*Email : *sara@tidepool.org\r\n*Web : *Tidepool.org <http://tidepool.org/>\r\n\r\n\r\n\r\n\r\nOn Mon, Jan 12, 2015 at 4:16 PM, Jana Beck <notifications@github.com> wrote:\r\n\r\n> Whether your very dark purple passes depends on the background color\r\n> @skrugman <https://github.com/skrugman> (I dunno whether you're still\r\n> talking about the day selectors for trends view...?)\r\n>\r\n> The contrast checker is here: http://webaim.org/resources/contrastchecker/\r\n>\r\n> Reminder: our guidelines are to meet WCAG at the AA level. Text has to be\r\n> at least 14px to count as \"large\".\r\n>\r\n> I will commit the darker color and push it up into this PR.\r\n>\r\n> —\r\n> Reply to this email directly or view it on GitHub\r\n> <https://github.com/tidepool-org/blip/pull/185#issuecomment-69673325>.\r\n>"}}