{"action":"created","comment":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/comments/99611066","pull_request_review_id":20292418,"id":99611066,"diff_hunk":"@@ -0,0 +1,196 @@\n+﻿using System;\n+using System.Collections.Generic;\n+using System.Collections.ObjectModel;\n+using System.Globalization;\n+using System.Xml;\n+using System.ComponentModel;\n+using System.Linq;\n+using System.Windows.Media;\n+using ProtoCore.AST.AssociativeAST;\n+using DSColor = DSCore.Color;\n+using Autodesk.DesignScript.Runtime;\n+using Dynamo.Graph.Nodes;\n+using Dynamo.Graph;\n+\n+using CoreNodeModels.Properties;\n+\n+namespace ColorNode\n+{\n+    /// <summary>\n+    ///     Custom Color UI node.\n+    /// </summary>\n+    [NodeName(\"Color Picker\")]\n+    [NodeDescription(\"ColorPaletteDescription\", typeof(Resources))]\n+    [NodeCategory(\"Core.Color.Create\")]\n+    [NodeSearchTags(\"ColorPaletteSearchTags\", typeof(Resources))]\n+    [IsDesignScriptCompatible]\n+    [OutPortNames(\"Color\")]\n+    [OutPortTypes(\"Color\")]\n+    [OutPortDescriptions(\"Selected Color.\")]\n+    public class nColorPicker : NodeModel, INotifyPropertyChanged","path":"src/Libraries/CoreNodeModels/ColorPalette.cs","position":30,"original_position":30,"commit_id":"c7a94625a79fbb4d5519046a232c62a7941bbb68","original_commit_id":"c7a94625a79fbb4d5519046a232c62a7941bbb68","user":{"login":"mjkkirschner","id":508936,"avatar_url":"https://avatars.githubusercontent.com/u/508936?v=3","gravatar_id":"","url":"https://api.github.com/users/mjkkirschner","html_url":"https://github.com/mjkkirschner","followers_url":"https://api.github.com/users/mjkkirschner/followers","following_url":"https://api.github.com/users/mjkkirschner/following{/other_user}","gists_url":"https://api.github.com/users/mjkkirschner/gists{/gist_id}","starred_url":"https://api.github.com/users/mjkkirschner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mjkkirschner/subscriptions","organizations_url":"https://api.github.com/users/mjkkirschner/orgs","repos_url":"https://api.github.com/users/mjkkirschner/repos","events_url":"https://api.github.com/users/mjkkirschner/events{/privacy}","received_events_url":"https://api.github.com/users/mjkkirschner/received_events","type":"User","site_admin":false},"body":"I believe NodeModel already does implement INotifyPropertyChanged.","created_at":"2017-02-06T15:36:54Z","updated_at":"2017-02-06T15:36:54Z","html_url":"https://github.com/DynamoDS/Dynamo/pull/7580#discussion_r99611066","pull_request_url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580","_links":{"self":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/comments/99611066"},"html":{"href":"https://github.com/DynamoDS/Dynamo/pull/7580#discussion_r99611066"},"pull_request":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580"}}},"pull_request":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580","id":104745424,"html_url":"https://github.com/DynamoDS/Dynamo/pull/7580","diff_url":"https://github.com/DynamoDS/Dynamo/pull/7580.diff","patch_url":"https://github.com/DynamoDS/Dynamo/pull/7580.patch","issue_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/7580","number":7580,"state":"open","locked":false,"title":"Color Palette UI Node","user":{"login":"Gytaco","id":5660565,"avatar_url":"https://avatars.githubusercontent.com/u/5660565?v=3","gravatar_id":"","url":"https://api.github.com/users/Gytaco","html_url":"https://github.com/Gytaco","followers_url":"https://api.github.com/users/Gytaco/followers","following_url":"https://api.github.com/users/Gytaco/following{/other_user}","gists_url":"https://api.github.com/users/Gytaco/gists{/gist_id}","starred_url":"https://api.github.com/users/Gytaco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gytaco/subscriptions","organizations_url":"https://api.github.com/users/Gytaco/orgs","repos_url":"https://api.github.com/users/Gytaco/repos","events_url":"https://api.github.com/users/Gytaco/events{/privacy}","received_events_url":"https://api.github.com/users/Gytaco/received_events","type":"User","site_admin":false},"body":"### Purpose\r\n\r\nThis feature integrates the color picker ui node functionality from UI++ into the dynamo core.\r\nhttps://dynamopackages.com/ search for UI++\r\n\r\nThe functionality provides a palette node to pick a color from rather then typing in values.\r\n![image](https://cloud.githubusercontent.com/assets/5660565/22638663/4fd51d76-ec95-11e6-8072-73c6948ba763.png)\r\n\r\n![image](https://cloud.githubusercontent.com/assets/5660565/22640039/a6469b42-ec9c-11e6-84ff-dfe3768ef401.png)\r\n\r\n![image](https://cloud.githubusercontent.com/assets/5660565/22640061/bfd4502c-ec9c-11e6-9215-073015f7ac1c.png)\r\n\r\n### Impact assessment\r\n\r\n- [ ] Adds or changes a public interface (Add lukechurch to the reviewer list)\r\n- [x] API change is added to the [API Changes document](https://github.com/DynamoDS/Dynamo/wiki/API-Changes).\r\n- [x] Introduces new components (Add lukechurch to the reviewer list)\r\n- [x] Changes the installer (Add sharadkjaiswal to the reviewer list)\r\n- [ ] User facing strings, if any, are extracted into `*.resx` files\r\n- [ ] All tests pass using the self-service CI.\r\n- [x] Snapshot of UI changes, if any.\r\n\r\n#### References \r\n\r\n```\r\nDynamo\\src\\Libraries\\CoreNodeModels\\Input\\ColorPalette.cs\r\nDynamo\\src\\Libraries\\CoreNodeModelsWpf\\Controls\\ColorPalette.xaml\r\nDynamo\\src\\Libraries\\CoreNodeModelsWpf\\NodeViewCustomizations\\ColorPalette.cs\r\n```\r\nAddition of Extended.WPF.Toolkit v3.0 added as installed nuget package to CoreNodeModelsWpf project.\r\nReferencing the Xceed.Wpf.Toolkit.dll for the palette component from the package.\r\n\r\n#### Testing strategy\r\n\r\nTo be advised.\r\n\r\n### Review staging\r\n\r\n- [ ] UI Review\r\n\r\nLooks like the mono build is breaking I am going to assume this is based on using System.Windows.Media to access to bind to the color object in the WPF. Xceed toolkit requires the use of the System.Windows.Media Color class.\r\n```\r\nDynamo\\src\\Libraries\\CoreNodeModels\\Input\\ColorPalette.cs\r\n```\r\n\r\n### Reviewers\r\n\r\nTBA\r\n\r\n### FYIs\r\n\r\nTBA","created_at":"2017-02-06T07:07:08Z","updated_at":"2017-02-06T15:36:54Z","closed_at":null,"merged_at":null,"merge_commit_sha":"740a0c75f02033faee6f66d45b0c5c3273bc3004","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580/commits","review_comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580/comments","review_comment_url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/comments{/number}","comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/7580/comments","statuses_url":"https://api.github.com/repos/DynamoDS/Dynamo/statuses/c7a94625a79fbb4d5519046a232c62a7941bbb68","head":{"label":"Gytaco:master","ref":"master","sha":"c7a94625a79fbb4d5519046a232c62a7941bbb68","user":{"login":"Gytaco","id":5660565,"avatar_url":"https://avatars.githubusercontent.com/u/5660565?v=3","gravatar_id":"","url":"https://api.github.com/users/Gytaco","html_url":"https://github.com/Gytaco","followers_url":"https://api.github.com/users/Gytaco/followers","following_url":"https://api.github.com/users/Gytaco/following{/other_user}","gists_url":"https://api.github.com/users/Gytaco/gists{/gist_id}","starred_url":"https://api.github.com/users/Gytaco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gytaco/subscriptions","organizations_url":"https://api.github.com/users/Gytaco/orgs","repos_url":"https://api.github.com/users/Gytaco/repos","events_url":"https://api.github.com/users/Gytaco/events{/privacy}","received_events_url":"https://api.github.com/users/Gytaco/received_events","type":"User","site_admin":false},"repo":{"id":80967162,"name":"Dynamo","full_name":"Gytaco/Dynamo","owner":{"login":"Gytaco","id":5660565,"avatar_url":"https://avatars.githubusercontent.com/u/5660565?v=3","gravatar_id":"","url":"https://api.github.com/users/Gytaco","html_url":"https://github.com/Gytaco","followers_url":"https://api.github.com/users/Gytaco/followers","following_url":"https://api.github.com/users/Gytaco/following{/other_user}","gists_url":"https://api.github.com/users/Gytaco/gists{/gist_id}","starred_url":"https://api.github.com/users/Gytaco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gytaco/subscriptions","organizations_url":"https://api.github.com/users/Gytaco/orgs","repos_url":"https://api.github.com/users/Gytaco/repos","events_url":"https://api.github.com/users/Gytaco/events{/privacy}","received_events_url":"https://api.github.com/users/Gytaco/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/Gytaco/Dynamo","description":"Visual Programming","fork":true,"url":"https://api.github.com/repos/Gytaco/Dynamo","forks_url":"https://api.github.com/repos/Gytaco/Dynamo/forks","keys_url":"https://api.github.com/repos/Gytaco/Dynamo/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Gytaco/Dynamo/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Gytaco/Dynamo/teams","hooks_url":"https://api.github.com/repos/Gytaco/Dynamo/hooks","issue_events_url":"https://api.github.com/repos/Gytaco/Dynamo/issues/events{/number}","events_url":"https://api.github.com/repos/Gytaco/Dynamo/events","assignees_url":"https://api.github.com/repos/Gytaco/Dynamo/assignees{/user}","branches_url":"https://api.github.com/repos/Gytaco/Dynamo/branches{/branch}","tags_url":"https://api.github.com/repos/Gytaco/Dynamo/tags","blobs_url":"https://api.github.com/repos/Gytaco/Dynamo/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Gytaco/Dynamo/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Gytaco/Dynamo/git/refs{/sha}","trees_url":"https://api.github.com/repos/Gytaco/Dynamo/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Gytaco/Dynamo/statuses/{sha}","languages_url":"https://api.github.com/repos/Gytaco/Dynamo/languages","stargazers_url":"https://api.github.com/repos/Gytaco/Dynamo/stargazers","contributors_url":"https://api.github.com/repos/Gytaco/Dynamo/contributors","subscribers_url":"https://api.github.com/repos/Gytaco/Dynamo/subscribers","subscription_url":"https://api.github.com/repos/Gytaco/Dynamo/subscription","commits_url":"https://api.github.com/repos/Gytaco/Dynamo/commits{/sha}","git_commits_url":"https://api.github.com/repos/Gytaco/Dynamo/git/commits{/sha}","comments_url":"https://api.github.com/repos/Gytaco/Dynamo/comments{/number}","issue_comment_url":"https://api.github.com/repos/Gytaco/Dynamo/issues/comments{/number}","contents_url":"https://api.github.com/repos/Gytaco/Dynamo/contents/{+path}","compare_url":"https://api.github.com/repos/Gytaco/Dynamo/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Gytaco/Dynamo/merges","archive_url":"https://api.github.com/repos/Gytaco/Dynamo/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Gytaco/Dynamo/downloads","issues_url":"https://api.github.com/repos/Gytaco/Dynamo/issues{/number}","pulls_url":"https://api.github.com/repos/Gytaco/Dynamo/pulls{/number}","milestones_url":"https://api.github.com/repos/Gytaco/Dynamo/milestones{/number}","notifications_url":"https://api.github.com/repos/Gytaco/Dynamo/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Gytaco/Dynamo/labels{/name}","releases_url":"https://api.github.com/repos/Gytaco/Dynamo/releases{/id}","deployments_url":"https://api.github.com/repos/Gytaco/Dynamo/deployments","created_at":"2017-02-05T03:52:57Z","updated_at":"2017-02-05T03:53:13Z","pushed_at":"2017-02-06T07:07:05Z","git_url":"git://github.com/Gytaco/Dynamo.git","ssh_url":"git@github.com:Gytaco/Dynamo.git","clone_url":"https://github.com/Gytaco/Dynamo.git","svn_url":"https://github.com/Gytaco/Dynamo","homepage":"http://dynamobim.org","size":978448,"stargazers_count":0,"watchers_count":0,"language":"C#","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":0,"default_branch":"master"}},"base":{"label":"DynamoDS:master","ref":"master","sha":"9856b8386b41056852a18f37ea47cdb274a78435","user":{"login":"DynamoDS","id":6528209,"avatar_url":"https://avatars.githubusercontent.com/u/6528209?v=3","gravatar_id":"","url":"https://api.github.com/users/DynamoDS","html_url":"https://github.com/DynamoDS","followers_url":"https://api.github.com/users/DynamoDS/followers","following_url":"https://api.github.com/users/DynamoDS/following{/other_user}","gists_url":"https://api.github.com/users/DynamoDS/gists{/gist_id}","starred_url":"https://api.github.com/users/DynamoDS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DynamoDS/subscriptions","organizations_url":"https://api.github.com/users/DynamoDS/orgs","repos_url":"https://api.github.com/users/DynamoDS/repos","events_url":"https://api.github.com/users/DynamoDS/events{/privacy}","received_events_url":"https://api.github.com/users/DynamoDS/received_events","type":"Organization","site_admin":false},"repo":{"id":3395179,"name":"Dynamo","full_name":"DynamoDS/Dynamo","owner":{"login":"DynamoDS","id":6528209,"avatar_url":"https://avatars.githubusercontent.com/u/6528209?v=3","gravatar_id":"","url":"https://api.github.com/users/DynamoDS","html_url":"https://github.com/DynamoDS","followers_url":"https://api.github.com/users/DynamoDS/followers","following_url":"https://api.github.com/users/DynamoDS/following{/other_user}","gists_url":"https://api.github.com/users/DynamoDS/gists{/gist_id}","starred_url":"https://api.github.com/users/DynamoDS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DynamoDS/subscriptions","organizations_url":"https://api.github.com/users/DynamoDS/orgs","repos_url":"https://api.github.com/users/DynamoDS/repos","events_url":"https://api.github.com/users/DynamoDS/events{/privacy}","received_events_url":"https://api.github.com/users/DynamoDS/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/DynamoDS/Dynamo","description":"Visual Programming","fork":false,"url":"https://api.github.com/repos/DynamoDS/Dynamo","forks_url":"https://api.github.com/repos/DynamoDS/Dynamo/forks","keys_url":"https://api.github.com/repos/DynamoDS/Dynamo/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DynamoDS/Dynamo/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DynamoDS/Dynamo/teams","hooks_url":"https://api.github.com/repos/DynamoDS/Dynamo/hooks","issue_events_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/events{/number}","events_url":"https://api.github.com/repos/DynamoDS/Dynamo/events","assignees_url":"https://api.github.com/repos/DynamoDS/Dynamo/assignees{/user}","branches_url":"https://api.github.com/repos/DynamoDS/Dynamo/branches{/branch}","tags_url":"https://api.github.com/repos/DynamoDS/Dynamo/tags","blobs_url":"https://api.github.com/repos/DynamoDS/Dynamo/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DynamoDS/Dynamo/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DynamoDS/Dynamo/git/refs{/sha}","trees_url":"https://api.github.com/repos/DynamoDS/Dynamo/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DynamoDS/Dynamo/statuses/{sha}","languages_url":"https://api.github.com/repos/DynamoDS/Dynamo/languages","stargazers_url":"https://api.github.com/repos/DynamoDS/Dynamo/stargazers","contributors_url":"https://api.github.com/repos/DynamoDS/Dynamo/contributors","subscribers_url":"https://api.github.com/repos/DynamoDS/Dynamo/subscribers","subscription_url":"https://api.github.com/repos/DynamoDS/Dynamo/subscription","commits_url":"https://api.github.com/repos/DynamoDS/Dynamo/commits{/sha}","git_commits_url":"https://api.github.com/repos/DynamoDS/Dynamo/git/commits{/sha}","comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/comments{/number}","issue_comment_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/comments{/number}","contents_url":"https://api.github.com/repos/DynamoDS/Dynamo/contents/{+path}","compare_url":"https://api.github.com/repos/DynamoDS/Dynamo/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DynamoDS/Dynamo/merges","archive_url":"https://api.github.com/repos/DynamoDS/Dynamo/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DynamoDS/Dynamo/downloads","issues_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues{/number}","pulls_url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls{/number}","milestones_url":"https://api.github.com/repos/DynamoDS/Dynamo/milestones{/number}","notifications_url":"https://api.github.com/repos/DynamoDS/Dynamo/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DynamoDS/Dynamo/labels{/name}","releases_url":"https://api.github.com/repos/DynamoDS/Dynamo/releases{/id}","deployments_url":"https://api.github.com/repos/DynamoDS/Dynamo/deployments","created_at":"2012-02-09T07:35:33Z","updated_at":"2017-02-02T12:29:54Z","pushed_at":"2017-02-06T08:58:53Z","git_url":"git://github.com/DynamoDS/Dynamo.git","ssh_url":"git@github.com:DynamoDS/Dynamo.git","clone_url":"https://github.com/DynamoDS/Dynamo.git","svn_url":"https://github.com/DynamoDS/Dynamo","homepage":"http://dynamobim.org","size":977735,"stargazers_count":414,"watchers_count":414,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":282,"mirror_url":null,"open_issues_count":710,"forks":282,"open_issues":710,"watchers":414,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580"},"html":{"href":"https://github.com/DynamoDS/Dynamo/pull/7580"},"issue":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/issues/7580"},"comments":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/issues/7580/comments"},"review_comments":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580/comments"},"review_comment":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/7580/commits"},"statuses":{"href":"https://api.github.com/repos/DynamoDS/Dynamo/statuses/c7a94625a79fbb4d5519046a232c62a7941bbb68"}}}}
{"actor":{"display_login":"mjkkirschner"}}