{"action":"created","comment":{"url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/comments/55790086","id":55790086,"diff_hunk":"@@ -0,0 +1,71 @@\n+/**\n+ * External dependencies\n+ */\n+import React from 'react';\n+import noop from 'lodash/noop';\n+\n+/**\n+ * Internal dependencies\n+ */\n+import ThemesList from 'components/themes-list';\n+\n+const demoThemes = [ {\n+\tid: 'twentyfourteen',\n+\tname: 'Twenty Fourteen',\n+\tscreenshot: '//i1.wp.com/theme.wordpress.com/wp-content/themes/pub/twentyfourteen/screenshot.png',\n+\tactionLabel: 'Click Action Theme 1',\n+},\n+{\tid: 'twentyfifteen',\n+\tname: 'Twenty Fifteen',\n+\tscreenshot: '//i1.wp.com/theme.wordpress.com/wp-content/themes/pub/twentyfifteen/screenshot.png',\n+\tactionLabel: 'Click Action Theme 2',\n+},\n+{\tid: 'twentysixteen',\n+\tname: 'Twenty Sixteen',\n+\tscreenshot: '//i1.wp.com/theme.wordpress.com/wp-content/themes/pub/twentysixteen/screenshot.png',\n+\tactionLabel: 'Click Action Theme 3',\n+} ];\n+\n+export default React.createClass( {\n+\tdisplayName: 'ThemesListExample',\n+\n+\tgetActionLabel: theme => theme.actionLabel,\n+\n+\tgetButtonOptions() {\n+\t\treturn {\n+\t\t\taction1: {\n+\t\t\t\tlabel: 'Menu Item 1',\n+\t\t\t\taction: function() {\n+\t\t\t\t\tconsole.log( 'Menu Item 1 selected' );\n+\t\t\t\t}\n+\t\t\t},\n+\t\t\taction2: {\n+\t\t\t\tlabel: 'Menu Item 2',\n+\t\t\t\taction: function() {\n+\t\t\t\t\tconsole.log( 'Menu Item 2 selected' );\n+\t\t\t\t}\n+\t\t\t}\n+\t\t}\n+\t},\n+\n+\tonScreenshotClickFn( theme, index ) {\n+\t\tconsole.log( `Theme ${ theme.id } at ${ index } clicked` );\n+\t},","path":"client/components/themes-list/docs/example.jsx","position":53,"original_position":53,"commit_id":"43387a1cc1e2997c52b82c122a90b5da47273b4a","original_commit_id":"43387a1cc1e2997c52b82c122a90b5da47273b4a","user":{"login":"afsheenirani","id":14303849,"avatar_url":"https://avatars.githubusercontent.com/u/14303849?v=3","gravatar_id":"","url":"https://api.github.com/users/afsheenirani","html_url":"https://github.com/afsheenirani","followers_url":"https://api.github.com/users/afsheenirani/followers","following_url":"https://api.github.com/users/afsheenirani/following{/other_user}","gists_url":"https://api.github.com/users/afsheenirani/gists{/gist_id}","starred_url":"https://api.github.com/users/afsheenirani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afsheenirani/subscriptions","organizations_url":"https://api.github.com/users/afsheenirani/orgs","repos_url":"https://api.github.com/users/afsheenirani/repos","events_url":"https://api.github.com/users/afsheenirani/events{/privacy}","received_events_url":"https://api.github.com/users/afsheenirani/received_events","type":"User","site_admin":false},"body":"Feedback duly noted. I did find it confusing to deal with `onScreenShotClick` of `ThemesList`, `onScreenShotClick` of `Theme` and then `onScreenShotClickFn` as the func to `ThemesList`. \r\n\r\nI will definitely incorporate this suggestion to avoid confusion around func prop naming.","created_at":"2016-03-11T05:10:03Z","updated_at":"2016-03-11T05:10:03Z","html_url":"https://github.com/Automattic/wp-calypso/pull/3625#discussion_r55790086","pull_request_url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625","_links":{"self":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/comments/55790086"},"html":{"href":"https://github.com/Automattic/wp-calypso/pull/3625#discussion_r55790086"},"pull_request":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625"}}},"pull_request":{"url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625","id":60940287,"html_url":"https://github.com/Automattic/wp-calypso/pull/3625","diff_url":"https://github.com/Automattic/wp-calypso/pull/3625.diff","patch_url":"https://github.com/Automattic/wp-calypso/pull/3625.patch","issue_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/3625","number":3625,"state":"open","locked":false,"title":"DevDocs\\Theme: Add ThemesList example app-component","user":{"login":"afsheenirani","id":14303849,"avatar_url":"https://avatars.githubusercontent.com/u/14303849?v=3","gravatar_id":"","url":"https://api.github.com/users/afsheenirani","html_url":"https://github.com/afsheenirani","followers_url":"https://api.github.com/users/afsheenirani/followers","following_url":"https://api.github.com/users/afsheenirani/following{/other_user}","gists_url":"https://api.github.com/users/afsheenirani/gists{/gist_id}","starred_url":"https://api.github.com/users/afsheenirani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afsheenirani/subscriptions","organizations_url":"https://api.github.com/users/afsheenirani/orgs","repos_url":"https://api.github.com/users/afsheenirani/repos","events_url":"https://api.github.com/users/afsheenirani/events{/privacy}","received_events_url":"https://api.github.com/users/afsheenirani/received_events","type":"User","site_admin":false},"body":"## Background\r\n\r\n- Add the `ThemesList` application component example to the [devdocs\\design\\app-components](https://wpcalypso.wordpress.com/devdocs/app-components) page.\r\n- Also, make the list of application components in alphabetical in order to have a better visual experience when applying search filtering at the top of the page.\r\n\r\n## Testing\r\n1. Load the [app-components](http://calypso.localhost:3000/devdocs/app-components) page.\r\n2. Scroll down to ensure you see the `Themes List` example.\r\n\r\n![image](https://cloud.githubusercontent.com/assets/14303849/13371509/ef8ae734-dcf5-11e5-8d94-f58e55976239.png)\r\n3. Search for `Themes` in the search box at the top of the page to ensure filtering works.\r\n4. Click on the `Themes List` link to make sure it takes you to a page with ONLY the `Themes List` component i.e., [app-components/themes-list](http://calypso.localhost:3000/devdocs/app-components/themes-list).\r\n5. Hover over each of the three Themes in the `ThemesList` component. Make sure this shows you individual Action labels for each theme e.g. `CLICK ACTION THEME 1` or `CLICK ACTION THEME 2`.\r\n6. Click on each theme and view the developer console logs to make sure it prints individual messages for each theme e.g., `Theme twentyfourteen at 0 clicked`. This tells you the theme id and index of that theme in the `ThemesList` that was clicked.\r\n7. Click on the More Buttons `...` link on any of the themes to make sure they work and when you click one of the links that pop-up you should see a message in the Developer console as `Menu Item 1 selected`.\r\n\r\n## Reviewers\r\n@designsimply, @nb, @folletto, @ockham \r\n\r\n## Issues\r\nCloses #2119 ","created_at":"2016-02-27T07:11:00Z","updated_at":"2016-03-11T05:10:03Z","closed_at":null,"merged_at":null,"merge_commit_sha":"930edc31167151c36d9079df49e305dfb9ab5013","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625/commits","review_comments_url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625/comments","review_comment_url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/3625/comments","statuses_url":"https://api.github.com/repos/Automattic/wp-calypso/statuses/43387a1cc1e2997c52b82c122a90b5da47273b4a","head":{"label":"afsheenirani:add/devdocs-themelist-component","ref":"add/devdocs-themelist-component","sha":"43387a1cc1e2997c52b82c122a90b5da47273b4a","user":{"login":"afsheenirani","id":14303849,"avatar_url":"https://avatars.githubusercontent.com/u/14303849?v=3","gravatar_id":"","url":"https://api.github.com/users/afsheenirani","html_url":"https://github.com/afsheenirani","followers_url":"https://api.github.com/users/afsheenirani/followers","following_url":"https://api.github.com/users/afsheenirani/following{/other_user}","gists_url":"https://api.github.com/users/afsheenirani/gists{/gist_id}","starred_url":"https://api.github.com/users/afsheenirani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afsheenirani/subscriptions","organizations_url":"https://api.github.com/users/afsheenirani/orgs","repos_url":"https://api.github.com/users/afsheenirani/repos","events_url":"https://api.github.com/users/afsheenirani/events{/privacy}","received_events_url":"https://api.github.com/users/afsheenirani/received_events","type":"User","site_admin":false},"repo":{"id":52487474,"name":"wp-calypso","full_name":"afsheenirani/wp-calypso","owner":{"login":"afsheenirani","id":14303849,"avatar_url":"https://avatars.githubusercontent.com/u/14303849?v=3","gravatar_id":"","url":"https://api.github.com/users/afsheenirani","html_url":"https://github.com/afsheenirani","followers_url":"https://api.github.com/users/afsheenirani/followers","following_url":"https://api.github.com/users/afsheenirani/following{/other_user}","gists_url":"https://api.github.com/users/afsheenirani/gists{/gist_id}","starred_url":"https://api.github.com/users/afsheenirani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afsheenirani/subscriptions","organizations_url":"https://api.github.com/users/afsheenirani/orgs","repos_url":"https://api.github.com/users/afsheenirani/repos","events_url":"https://api.github.com/users/afsheenirani/events{/privacy}","received_events_url":"https://api.github.com/users/afsheenirani/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/afsheenirani/wp-calypso","description":"The new JavaScript- and API-powered WordPress.com","fork":true,"url":"https://api.github.com/repos/afsheenirani/wp-calypso","forks_url":"https://api.github.com/repos/afsheenirani/wp-calypso/forks","keys_url":"https://api.github.com/repos/afsheenirani/wp-calypso/keys{/key_id}","collaborators_url":"https://api.github.com/repos/afsheenirani/wp-calypso/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/afsheenirani/wp-calypso/teams","hooks_url":"https://api.github.com/repos/afsheenirani/wp-calypso/hooks","issue_events_url":"https://api.github.com/repos/afsheenirani/wp-calypso/issues/events{/number}","events_url":"https://api.github.com/repos/afsheenirani/wp-calypso/events","assignees_url":"https://api.github.com/repos/afsheenirani/wp-calypso/assignees{/user}","branches_url":"https://api.github.com/repos/afsheenirani/wp-calypso/branches{/branch}","tags_url":"https://api.github.com/repos/afsheenirani/wp-calypso/tags","blobs_url":"https://api.github.com/repos/afsheenirani/wp-calypso/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/afsheenirani/wp-calypso/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/afsheenirani/wp-calypso/git/refs{/sha}","trees_url":"https://api.github.com/repos/afsheenirani/wp-calypso/git/trees{/sha}","statuses_url":"https://api.github.com/repos/afsheenirani/wp-calypso/statuses/{sha}","languages_url":"https://api.github.com/repos/afsheenirani/wp-calypso/languages","stargazers_url":"https://api.github.com/repos/afsheenirani/wp-calypso/stargazers","contributors_url":"https://api.github.com/repos/afsheenirani/wp-calypso/contributors","subscribers_url":"https://api.github.com/repos/afsheenirani/wp-calypso/subscribers","subscription_url":"https://api.github.com/repos/afsheenirani/wp-calypso/subscription","commits_url":"https://api.github.com/repos/afsheenirani/wp-calypso/commits{/sha}","git_commits_url":"https://api.github.com/repos/afsheenirani/wp-calypso/git/commits{/sha}","comments_url":"https://api.github.com/repos/afsheenirani/wp-calypso/comments{/number}","issue_comment_url":"https://api.github.com/repos/afsheenirani/wp-calypso/issues/comments{/number}","contents_url":"https://api.github.com/repos/afsheenirani/wp-calypso/contents/{+path}","compare_url":"https://api.github.com/repos/afsheenirani/wp-calypso/compare/{base}...{head}","merges_url":"https://api.github.com/repos/afsheenirani/wp-calypso/merges","archive_url":"https://api.github.com/repos/afsheenirani/wp-calypso/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/afsheenirani/wp-calypso/downloads","issues_url":"https://api.github.com/repos/afsheenirani/wp-calypso/issues{/number}","pulls_url":"https://api.github.com/repos/afsheenirani/wp-calypso/pulls{/number}","milestones_url":"https://api.github.com/repos/afsheenirani/wp-calypso/milestones{/number}","notifications_url":"https://api.github.com/repos/afsheenirani/wp-calypso/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/afsheenirani/wp-calypso/labels{/name}","releases_url":"https://api.github.com/repos/afsheenirani/wp-calypso/releases{/id}","deployments_url":"https://api.github.com/repos/afsheenirani/wp-calypso/deployments","created_at":"2016-02-25T01:31:30Z","updated_at":"2016-03-02T05:38:38Z","pushed_at":"2016-03-10T01:24:58Z","git_url":"git://github.com/afsheenirani/wp-calypso.git","ssh_url":"git@github.com:afsheenirani/wp-calypso.git","clone_url":"https://github.com/afsheenirani/wp-calypso.git","svn_url":"https://github.com/afsheenirani/wp-calypso","homepage":"https://developer.wordpress.com/calypso/","size":13928,"stargazers_count":1,"watchers_count":1,"language":"JavaScript","has_issues":false,"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":1,"default_branch":"master"}},"base":{"label":"Automattic:master","ref":"master","sha":"55087a723cb7d653b3fe8a984770a8b70f3564c3","user":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"repo":{"id":45936895,"name":"wp-calypso","full_name":"Automattic/wp-calypso","owner":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Automattic/wp-calypso","description":"The new JavaScript- and API-powered WordPress.com","fork":false,"url":"https://api.github.com/repos/Automattic/wp-calypso","forks_url":"https://api.github.com/repos/Automattic/wp-calypso/forks","keys_url":"https://api.github.com/repos/Automattic/wp-calypso/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Automattic/wp-calypso/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Automattic/wp-calypso/teams","hooks_url":"https://api.github.com/repos/Automattic/wp-calypso/hooks","issue_events_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/events{/number}","events_url":"https://api.github.com/repos/Automattic/wp-calypso/events","assignees_url":"https://api.github.com/repos/Automattic/wp-calypso/assignees{/user}","branches_url":"https://api.github.com/repos/Automattic/wp-calypso/branches{/branch}","tags_url":"https://api.github.com/repos/Automattic/wp-calypso/tags","blobs_url":"https://api.github.com/repos/Automattic/wp-calypso/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Automattic/wp-calypso/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Automattic/wp-calypso/git/refs{/sha}","trees_url":"https://api.github.com/repos/Automattic/wp-calypso/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Automattic/wp-calypso/statuses/{sha}","languages_url":"https://api.github.com/repos/Automattic/wp-calypso/languages","stargazers_url":"https://api.github.com/repos/Automattic/wp-calypso/stargazers","contributors_url":"https://api.github.com/repos/Automattic/wp-calypso/contributors","subscribers_url":"https://api.github.com/repos/Automattic/wp-calypso/subscribers","subscription_url":"https://api.github.com/repos/Automattic/wp-calypso/subscription","commits_url":"https://api.github.com/repos/Automattic/wp-calypso/commits{/sha}","git_commits_url":"https://api.github.com/repos/Automattic/wp-calypso/git/commits{/sha}","comments_url":"https://api.github.com/repos/Automattic/wp-calypso/comments{/number}","issue_comment_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/comments{/number}","contents_url":"https://api.github.com/repos/Automattic/wp-calypso/contents/{+path}","compare_url":"https://api.github.com/repos/Automattic/wp-calypso/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Automattic/wp-calypso/merges","archive_url":"https://api.github.com/repos/Automattic/wp-calypso/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Automattic/wp-calypso/downloads","issues_url":"https://api.github.com/repos/Automattic/wp-calypso/issues{/number}","pulls_url":"https://api.github.com/repos/Automattic/wp-calypso/pulls{/number}","milestones_url":"https://api.github.com/repos/Automattic/wp-calypso/milestones{/number}","notifications_url":"https://api.github.com/repos/Automattic/wp-calypso/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Automattic/wp-calypso/labels{/name}","releases_url":"https://api.github.com/repos/Automattic/wp-calypso/releases{/id}","deployments_url":"https://api.github.com/repos/Automattic/wp-calypso/deployments","created_at":"2015-11-10T20:00:16Z","updated_at":"2016-03-11T02:58:43Z","pushed_at":"2016-03-11T03:14:47Z","git_url":"git://github.com/Automattic/wp-calypso.git","ssh_url":"git@github.com:Automattic/wp-calypso.git","clone_url":"https://github.com/Automattic/wp-calypso.git","svn_url":"https://github.com/Automattic/wp-calypso","homepage":"https://developer.wordpress.com/calypso/","size":56804,"stargazers_count":6960,"watchers_count":6960,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":807,"mirror_url":null,"open_issues_count":1313,"forks":807,"open_issues":1313,"watchers":6960,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625"},"html":{"href":"https://github.com/Automattic/wp-calypso/pull/3625"},"issue":{"href":"https://api.github.com/repos/Automattic/wp-calypso/issues/3625"},"comments":{"href":"https://api.github.com/repos/Automattic/wp-calypso/issues/3625/comments"},"review_comments":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625/comments"},"review_comment":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Automattic/wp-calypso/pulls/3625/commits"},"statuses":{"href":"https://api.github.com/repos/Automattic/wp-calypso/statuses/43387a1cc1e2997c52b82c122a90b5da47273b4a"}}}}