{"action":"created","comment":{"url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/comments/130430840","pull_request_review_id":53305806,"id":130430840,"diff_hunk":"@@ -5,21 +5,21 @@\n \n ## Introduction\n \n-Matroska supports storage of related files and data in the Attachments Top-Level Element. Attachments can be used to store related cover art, font files, transcripts, reports, error recovery files, picture or text-based annotations, copies of specifications, or other ancilliary files related to the Segment.\n+Matroska supports storage of related files and data in the `Attachments Element` (a `Top-Level Element`). Attachments can be used to store related cover art, font files, transcripts, reports, error recovery files, picture or text-based annotations, copies of specifications, or other ancillary files related to the `Segment`.\n \n `Matroska Readers` MUST NOT execute files stored as Attachments.\n \n ## Cover Art\n \n-This section defines a set of guidelines for the storage of cover art in Matroska files. A `Matroska Reader` MAY use embedded cover art to display a representation still-image depiction of the multimedia contents of the Matroska file.\n+This section defines a set of guidelines for the storage of cover art in Matroska files. A `Matroska Reader` MAY use embedded cover art to display a representational still-image depiction of the multimedia contents of the Matroska file.\n \n-Cover art SHOULD only use the JPEG and PNG picture formats.\n+Cover art SHOULD only use JPEG and PNG formats.\n \n-There can be 2 different covers for a movie/album. A portrait one (like a DVD case) and a landscape one (like a banner ad for example, looking better on a wide screen).\n+There can be 2 different covers for a movie/album. A portrait style (e.g., a DVD case) and a landscape style (e.g., a wide banner ad).\n \n-There can be 2 versions of the same cover, the `normal cover` and the `small cover`. The dimension of the `normal cover` SHOULD be 600 on the smallest side (for example, 960x600 for landscape, 600x800 for portrait, or 600x600 for square). The dimension of the `small cover` SHOULD be 120 on the smallest side (for example, 192x120 or 120x160).\n+There can be 2 versions of the same cover, the `normal cover` and the `small cover`. The dimension of the `normal cover` SHOULD be 600 pixels on the smallest side (for example, 960x600 for landscape, 600x800 for portrait, or 600x600 for square). The dimension of the `small cover` SHOULD be 120 pixels on the smallest side (for example, 192x120 or 120x160).","path":"attachments.md","position":21,"original_position":21,"commit_id":"bc5da33486cd1d4e4f680bfd9f5041e833dd2ae9","original_commit_id":"bc5da33486cd1d4e4f680bfd9f5041e833dd2ae9","user":{"login":"retokromer","id":3246607,"avatar_url":"https://avatars2.githubusercontent.com/u/3246607?v=4","gravatar_id":"","url":"https://api.github.com/users/retokromer","html_url":"https://github.com/retokromer","followers_url":"https://api.github.com/users/retokromer/followers","following_url":"https://api.github.com/users/retokromer/following{/other_user}","gists_url":"https://api.github.com/users/retokromer/gists{/gist_id}","starred_url":"https://api.github.com/users/retokromer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/retokromer/subscriptions","organizations_url":"https://api.github.com/users/retokromer/orgs","repos_url":"https://api.github.com/users/retokromer/repos","events_url":"https://api.github.com/users/retokromer/events{/privacy}","received_events_url":"https://api.github.com/users/retokromer/received_events","type":"User","site_admin":false},"body":"(It’s the same in French, Italian and Romansh.)","created_at":"2017-07-31T18:52:50Z","updated_at":"2017-07-31T18:52:50Z","html_url":"https://github.com/Matroska-Org/matroska-specification/pull/159#discussion_r130430840","pull_request_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159","_links":{"self":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/comments/130430840"},"html":{"href":"https://github.com/Matroska-Org/matroska-specification/pull/159#discussion_r130430840"},"pull_request":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159"}}},"pull_request":{"url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159","id":133148875,"html_url":"https://github.com/Matroska-Org/matroska-specification/pull/159","diff_url":"https://github.com/Matroska-Org/matroska-specification/pull/159.diff","patch_url":"https://github.com/Matroska-Org/matroska-specification/pull/159.patch","issue_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/159","number":159,"state":"open","locked":false,"title":"cleans up syntax and style for attachments.md","user":{"login":"ablwr","id":3260492,"avatar_url":"https://avatars2.githubusercontent.com/u/3260492?v=4","gravatar_id":"","url":"https://api.github.com/users/ablwr","html_url":"https://github.com/ablwr","followers_url":"https://api.github.com/users/ablwr/followers","following_url":"https://api.github.com/users/ablwr/following{/other_user}","gists_url":"https://api.github.com/users/ablwr/gists{/gist_id}","starred_url":"https://api.github.com/users/ablwr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ablwr/subscriptions","organizations_url":"https://api.github.com/users/ablwr/orgs","repos_url":"https://api.github.com/users/ablwr/repos","events_url":"https://api.github.com/users/ablwr/events{/privacy}","received_events_url":"https://api.github.com/users/ablwr/received_events","type":"User","site_admin":false},"body":"Attachments.md is pretty sparse. I removed the header for Font Files since there was nothing in it, but maybe I should ask the list for opinions and potential spec guidelines?","created_at":"2017-07-31T00:15:12Z","updated_at":"2017-07-31T18:52:50Z","closed_at":null,"merged_at":null,"merge_commit_sha":"710de3d857d26bfbf249fdf6eae16e4cbfa47aad","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159/commits","review_comments_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159/comments","review_comment_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/159/comments","statuses_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/statuses/bc5da33486cd1d4e4f680bfd9f5041e833dd2ae9","head":{"label":"ablwr:ab/attachments-shape-up","ref":"ab/attachments-shape-up","sha":"bc5da33486cd1d4e4f680bfd9f5041e833dd2ae9","user":{"login":"ablwr","id":3260492,"avatar_url":"https://avatars2.githubusercontent.com/u/3260492?v=4","gravatar_id":"","url":"https://api.github.com/users/ablwr","html_url":"https://github.com/ablwr","followers_url":"https://api.github.com/users/ablwr/followers","following_url":"https://api.github.com/users/ablwr/following{/other_user}","gists_url":"https://api.github.com/users/ablwr/gists{/gist_id}","starred_url":"https://api.github.com/users/ablwr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ablwr/subscriptions","organizations_url":"https://api.github.com/users/ablwr/orgs","repos_url":"https://api.github.com/users/ablwr/repos","events_url":"https://api.github.com/users/ablwr/events{/privacy}","received_events_url":"https://api.github.com/users/ablwr/received_events","type":"User","site_admin":false},"repo":{"id":59066795,"name":"matroska-specification","full_name":"ablwr/matroska-specification","owner":{"login":"ablwr","id":3260492,"avatar_url":"https://avatars2.githubusercontent.com/u/3260492?v=4","gravatar_id":"","url":"https://api.github.com/users/ablwr","html_url":"https://github.com/ablwr","followers_url":"https://api.github.com/users/ablwr/followers","following_url":"https://api.github.com/users/ablwr/following{/other_user}","gists_url":"https://api.github.com/users/ablwr/gists{/gist_id}","starred_url":"https://api.github.com/users/ablwr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ablwr/subscriptions","organizations_url":"https://api.github.com/users/ablwr/orgs","repos_url":"https://api.github.com/users/ablwr/repos","events_url":"https://api.github.com/users/ablwr/events{/privacy}","received_events_url":"https://api.github.com/users/ablwr/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ablwr/matroska-specification","description":null,"fork":true,"url":"https://api.github.com/repos/ablwr/matroska-specification","forks_url":"https://api.github.com/repos/ablwr/matroska-specification/forks","keys_url":"https://api.github.com/repos/ablwr/matroska-specification/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ablwr/matroska-specification/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ablwr/matroska-specification/teams","hooks_url":"https://api.github.com/repos/ablwr/matroska-specification/hooks","issue_events_url":"https://api.github.com/repos/ablwr/matroska-specification/issues/events{/number}","events_url":"https://api.github.com/repos/ablwr/matroska-specification/events","assignees_url":"https://api.github.com/repos/ablwr/matroska-specification/assignees{/user}","branches_url":"https://api.github.com/repos/ablwr/matroska-specification/branches{/branch}","tags_url":"https://api.github.com/repos/ablwr/matroska-specification/tags","blobs_url":"https://api.github.com/repos/ablwr/matroska-specification/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ablwr/matroska-specification/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ablwr/matroska-specification/git/refs{/sha}","trees_url":"https://api.github.com/repos/ablwr/matroska-specification/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ablwr/matroska-specification/statuses/{sha}","languages_url":"https://api.github.com/repos/ablwr/matroska-specification/languages","stargazers_url":"https://api.github.com/repos/ablwr/matroska-specification/stargazers","contributors_url":"https://api.github.com/repos/ablwr/matroska-specification/contributors","subscribers_url":"https://api.github.com/repos/ablwr/matroska-specification/subscribers","subscription_url":"https://api.github.com/repos/ablwr/matroska-specification/subscription","commits_url":"https://api.github.com/repos/ablwr/matroska-specification/commits{/sha}","git_commits_url":"https://api.github.com/repos/ablwr/matroska-specification/git/commits{/sha}","comments_url":"https://api.github.com/repos/ablwr/matroska-specification/comments{/number}","issue_comment_url":"https://api.github.com/repos/ablwr/matroska-specification/issues/comments{/number}","contents_url":"https://api.github.com/repos/ablwr/matroska-specification/contents/{+path}","compare_url":"https://api.github.com/repos/ablwr/matroska-specification/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ablwr/matroska-specification/merges","archive_url":"https://api.github.com/repos/ablwr/matroska-specification/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ablwr/matroska-specification/downloads","issues_url":"https://api.github.com/repos/ablwr/matroska-specification/issues{/number}","pulls_url":"https://api.github.com/repos/ablwr/matroska-specification/pulls{/number}","milestones_url":"https://api.github.com/repos/ablwr/matroska-specification/milestones{/number}","notifications_url":"https://api.github.com/repos/ablwr/matroska-specification/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ablwr/matroska-specification/labels{/name}","releases_url":"https://api.github.com/repos/ablwr/matroska-specification/releases{/id}","deployments_url":"https://api.github.com/repos/ablwr/matroska-specification/deployments","created_at":"2016-05-17T23:40:05Z","updated_at":"2016-05-17T23:40:05Z","pushed_at":"2017-07-31T00:14:17Z","git_url":"git://github.com/ablwr/matroska-specification.git","ssh_url":"git@github.com:ablwr/matroska-specification.git","clone_url":"https://github.com/ablwr/matroska-specification.git","svn_url":"https://github.com/ablwr/matroska-specification","homepage":null,"size":473,"stargazers_count":0,"watchers_count":0,"language":"CSS","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"Matroska-Org:master","ref":"master","sha":"868f024cbd41559e0ef35fb42aa7343706bc3724","user":{"login":"Matroska-Org","id":3148284,"avatar_url":"https://avatars1.githubusercontent.com/u/3148284?v=4","gravatar_id":"","url":"https://api.github.com/users/Matroska-Org","html_url":"https://github.com/Matroska-Org","followers_url":"https://api.github.com/users/Matroska-Org/followers","following_url":"https://api.github.com/users/Matroska-Org/following{/other_user}","gists_url":"https://api.github.com/users/Matroska-Org/gists{/gist_id}","starred_url":"https://api.github.com/users/Matroska-Org/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Matroska-Org/subscriptions","organizations_url":"https://api.github.com/users/Matroska-Org/orgs","repos_url":"https://api.github.com/users/Matroska-Org/repos","events_url":"https://api.github.com/users/Matroska-Org/events{/privacy}","received_events_url":"https://api.github.com/users/Matroska-Org/received_events","type":"Organization","site_admin":false},"repo":{"id":56977388,"name":"matroska-specification","full_name":"Matroska-Org/matroska-specification","owner":{"login":"Matroska-Org","id":3148284,"avatar_url":"https://avatars1.githubusercontent.com/u/3148284?v=4","gravatar_id":"","url":"https://api.github.com/users/Matroska-Org","html_url":"https://github.com/Matroska-Org","followers_url":"https://api.github.com/users/Matroska-Org/followers","following_url":"https://api.github.com/users/Matroska-Org/following{/other_user}","gists_url":"https://api.github.com/users/Matroska-Org/gists{/gist_id}","starred_url":"https://api.github.com/users/Matroska-Org/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Matroska-Org/subscriptions","organizations_url":"https://api.github.com/users/Matroska-Org/orgs","repos_url":"https://api.github.com/users/Matroska-Org/repos","events_url":"https://api.github.com/users/Matroska-Org/events{/privacy}","received_events_url":"https://api.github.com/users/Matroska-Org/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Matroska-Org/matroska-specification","description":null,"fork":false,"url":"https://api.github.com/repos/Matroska-Org/matroska-specification","forks_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/forks","keys_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/teams","hooks_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/hooks","issue_events_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/events{/number}","events_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/events","assignees_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/assignees{/user}","branches_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/branches{/branch}","tags_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/tags","blobs_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/git/refs{/sha}","trees_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/statuses/{sha}","languages_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/languages","stargazers_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/stargazers","contributors_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/contributors","subscribers_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/subscribers","subscription_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/subscription","commits_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/commits{/sha}","git_commits_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/git/commits{/sha}","comments_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/comments{/number}","issue_comment_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/comments{/number}","contents_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/contents/{+path}","compare_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/merges","archive_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/downloads","issues_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues{/number}","pulls_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls{/number}","milestones_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/milestones{/number}","notifications_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/labels{/name}","releases_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/releases{/id}","deployments_url":"https://api.github.com/repos/Matroska-Org/matroska-specification/deployments","created_at":"2016-04-24T14:58:00Z","updated_at":"2017-07-15T06:14:37Z","pushed_at":"2017-07-31T00:15:13Z","git_url":"git://github.com/Matroska-Org/matroska-specification.git","ssh_url":"git@github.com:Matroska-Org/matroska-specification.git","clone_url":"https://github.com/Matroska-Org/matroska-specification.git","svn_url":"https://github.com/Matroska-Org/matroska-specification","homepage":null,"size":529,"stargazers_count":18,"watchers_count":18,"language":"XSLT","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":14,"mirror_url":null,"open_issues_count":36,"forks":14,"open_issues":36,"watchers":18,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159"},"html":{"href":"https://github.com/Matroska-Org/matroska-specification/pull/159"},"issue":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/159"},"comments":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/issues/159/comments"},"review_comments":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159/comments"},"review_comment":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/pulls/159/commits"},"statuses":{"href":"https://api.github.com/repos/Matroska-Org/matroska-specification/statuses/bc5da33486cd1d4e4f680bfd9f5041e833dd2ae9"}}}}
{"actor":{"display_login":"retokromer"}}