{"action":"created","issue":{"url":"https://api.github.com/repos/hypothesis/h/issues/3298","repository_url":"https://api.github.com/repos/hypothesis/h","labels_url":"https://api.github.com/repos/hypothesis/h/issues/3298/labels{/name}","comments_url":"https://api.github.com/repos/hypothesis/h/issues/3298/comments","events_url":"https://api.github.com/repos/hypothesis/h/issues/3298/events","html_url":"https://github.com/hypothesis/h/pull/3298","id":153944906,"number":3298,"title":"Display hovered conversations with a grey background","user":{"login":"robertknight","id":2458,"avatar_url":"https://avatars.githubusercontent.com/u/2458?v=3","gravatar_id":"","url":"https://api.github.com/users/robertknight","html_url":"https://github.com/robertknight","followers_url":"https://api.github.com/users/robertknight/followers","following_url":"https://api.github.com/users/robertknight/following{/other_user}","gists_url":"https://api.github.com/users/robertknight/gists{/gist_id}","starred_url":"https://api.github.com/users/robertknight/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/robertknight/subscriptions","organizations_url":"https://api.github.com/users/robertknight/orgs","repos_url":"https://api.github.com/users/robertknight/repos","events_url":"https://api.github.com/users/robertknight/events{/privacy}","received_events_url":"https://api.github.com/users/robertknight/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-05-10T07:56:38Z","updated_at":"2016-05-10T13:49:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hypothesis/h/pulls/3298","html_url":"https://github.com/hypothesis/h/pull/3298","diff_url":"https://github.com/hypothesis/h/pull/3298.diff","patch_url":"https://github.com/hypothesis/h/pull/3298.patch"},"body":"Implement the changes to the background hover style from\r\nhttps://trello.com/c/aXCXxzx2 and specifically this mock: https://trello-attachments.s3.amazonaws.com/571654d75b466478e53c0544/1373x901/1d1bfed40f039ddd84f41012a64c77e1/Annotation_card_refresh.png"},"comment":{"url":"https://api.github.com/repos/hypothesis/h/issues/comments/218162738","html_url":"https://github.com/hypothesis/h/pull/3298#issuecomment-218162738","issue_url":"https://api.github.com/repos/hypothesis/h/issues/3298","id":218162738,"user":{"login":"nickstenning","id":3602,"avatar_url":"https://avatars.githubusercontent.com/u/3602?v=3","gravatar_id":"","url":"https://api.github.com/users/nickstenning","html_url":"https://github.com/nickstenning","followers_url":"https://api.github.com/users/nickstenning/followers","following_url":"https://api.github.com/users/nickstenning/following{/other_user}","gists_url":"https://api.github.com/users/nickstenning/gists{/gist_id}","starred_url":"https://api.github.com/users/nickstenning/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickstenning/subscriptions","organizations_url":"https://api.github.com/users/nickstenning/orgs","repos_url":"https://api.github.com/users/nickstenning/repos","events_url":"https://api.github.com/users/nickstenning/events{/privacy}","received_events_url":"https://api.github.com/users/nickstenning/received_events","type":"User","site_admin":false},"created_at":"2016-05-10T13:49:25Z","updated_at":"2016-05-10T13:49:25Z","body":"Looking pretty good, but this is a big change. Do we perhaps want to feature flag this?\r\n\r\nAlso, couple of niggles.\r\n\r\n#### 1. Hovering over a collapsed reply shows \"blocky outcrop\"\r\n\r\nI'm guessing the rectangular block under the collapse indicator shouldn't be there?\r\n\r\n![screen shot 2016-05-10 at 15 44 56](https://cloud.githubusercontent.com/assets/3602/15148589/2ba2ba9e-16c6-11e6-99e6-04ba4c665b4a.png)\r\n\r\n#### 2. Hovering over one reply highlights all reply threads\r\n\r\nThe mockup shows each reply and its children being independently hoverable, but that's not how this PR works:\r\n\r\n![screen shot 2016-05-10 at 15 47 04](https://cloud.githubusercontent.com/assets/3602/15148667/9cf2d512-16c6-11e6-8cf7-fcfdf472820c.png)\r\n\r\n----\r\n\r\nAgain, in the interest of not having to merge this all at once and get it perfect first time, I'd be in favour of adding a feature flag which can be used to add a CSS selector, which in turn can be used to test this design before turning it on for everyone.\r\n\r\n"}}