GitHub PullRequestReviewCommentEvent at 2014-02-10 18:38:54
Repository: bedrock
Actor: jpetto
Payload: {"comment":{"url":"https://api.github.com/repos/mozilla/bedrock/pulls/comments/9595782","id":9595782,"diff_hunk":"@@ -43,28 +43,7 @@\n       {{ css('common') }}\n     {% endblock %}\n \n-    {% if LANG in ['ar', 'fa'] %}\n-    {# bug 958321 #}\n-      <style type=\"text/css\">\n-        @font-face {\n-          font-family: 'Droid Arabic Naskh';\n-          src: url({{ media('fonts/DroidNaskh-Regular.eot?#iefix') }}) format('embedded-opentype'),\n-               url({{ media('fonts/DroidNaskh-Regular.woff') }}) format('woff'),\n-               url({{ media('fonts/DroidNaskh-Regular.ttf') }}) format('truetype');\n-          font-style: normal;\n-          font-weight: normal;\n-        }\n-\n-        @font-face {\n-          font-family: 'Droid Arabic Naskh';\n-          src: url({{ media('/fonts/DroidNaskh-Bold.eot?#iefix') }}) format('embedded-opentype'),\n-               url({{ media('/fonts/DroidNaskh-Bold.woff') }}) format('woff'),\n-               url({{ media('/fonts/DroidNaskh-Bold.ttf') }}) format('truetype');\n-          font-style: normal;\n-          font-weight: bold;\n-        }\n-      </style>\n-    {% endif %}\n+    {% include 'includes/font-faces.html' %}","path":"bedrock/base/templates/base.html","position":26,"original_position":26,"commit_id":"daf9f42a6640f5d28d18a2dc734b638b009143eb","original_commit_id":"daf9f42a6640f5d28d18a2dc734b638b009143eb","user":{"login":"jpetto","id":317532,"avatar_url":"https://gravatar.com/avatar/58d5cdc21fe9f9dac826038e5398d963?d=https%3A%2F%2Fidenticons.github.com%2Fd2ea557d54c71c3bfbc1119b9467d5c5.png&r=x","gravatar_id":"58d5cdc21fe9f9dac826038e5398d963","url":"https://api.github.com/users/jpetto","html_url":"https://github.com/jpetto","followers_url":"https://api.github.com/users/jpetto/followers","following_url":"https://api.github.com/users/jpetto/following{/other_user}","gists_url":"https://api.github.com/users/jpetto/gists{/gist_id}","starred_url":"https://api.github.com/users/jpetto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpetto/subscriptions","organizations_url":"https://api.github.com/users/jpetto/orgs","repos_url":"https://api.github.com/users/jpetto/repos","events_url":"https://api.github.com/users/jpetto/events{/privacy}","received_events_url":"https://api.github.com/users/jpetto/received_events","type":"User","site_admin":false},"body":"I think `locale-font-faces.html` would be a better/more descriptive include name.","created_at":"2014-02-10T18:38:54Z","updated_at":"2014-02-10T18:38:54Z","html_url":"https://github.com/mozilla/bedrock/pull/1677#discussion_r9595782","pull_request_url":"https://api.github.com/repos/mozilla/bedrock/pulls/1677","_links":{"self":{"href":"https://api.github.com/repos/mozilla/bedrock/pulls/comments/9595782"},"html":{"href":"https://github.com/mozilla/bedrock/pull/1677#discussion_r9595782"},"pull_request":{"href":"https://api.github.com/repos/mozilla/bedrock/pulls/1677"}}}}