{"action":"created","issue":{"url":"https://api.github.com/repos/chartjs/Chart.js/issues/5741","repository_url":"https://api.github.com/repos/chartjs/Chart.js","labels_url":"https://api.github.com/repos/chartjs/Chart.js/issues/5741/labels{/name}","comments_url":"https://api.github.com/repos/chartjs/Chart.js/issues/5741/comments","events_url":"https://api.github.com/repos/chartjs/Chart.js/issues/5741/events","html_url":"https://github.com/chartjs/Chart.js/pull/5741","id":362196364,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE2OTc4Njc2","number":5741,"title":"Introduce the 'minBarLength' dataset property for bar scales (X or Y)","user":{"login":"adube","id":432436,"node_id":"MDQ6VXNlcjQzMjQzNg==","avatar_url":"https://avatars0.githubusercontent.com/u/432436?v=4","gravatar_id":"","url":"https://api.github.com/users/adube","html_url":"https://github.com/adube","followers_url":"https://api.github.com/users/adube/followers","following_url":"https://api.github.com/users/adube/following{/other_user}","gists_url":"https://api.github.com/users/adube/gists{/gist_id}","starred_url":"https://api.github.com/users/adube/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adube/subscriptions","organizations_url":"https://api.github.com/users/adube/orgs","repos_url":"https://api.github.com/users/adube/repos","events_url":"https://api.github.com/users/adube/events{/privacy}","received_events_url":"https://api.github.com/users/adube/received_events","type":"User","site_admin":false},"labels":[{"id":1022191787,"node_id":"MDU6TGFiZWwxMDIyMTkxNzg3","url":"https://api.github.com/repos/chartjs/Chart.js/labels/status:%20awaiting%20review","name":"status: awaiting review","color":"00b587","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/chartjs/Chart.js/milestones/18","html_url":"https://github.com/chartjs/Chart.js/milestone/18","labels_url":"https://api.github.com/repos/chartjs/Chart.js/milestones/18/labels","id":3657117,"node_id":"MDk6TWlsZXN0b25lMzY1NzExNw==","number":18,"title":"Version 2.8","description":"[Release Notes](/chartjs/Chart.js/releases/tag/v2.8)","creator":{"login":"simonbrunel","id":3874900,"node_id":"MDQ6VXNlcjM4NzQ5MDA=","avatar_url":"https://avatars2.githubusercontent.com/u/3874900?v=4","gravatar_id":"","url":"https://api.github.com/users/simonbrunel","html_url":"https://github.com/simonbrunel","followers_url":"https://api.github.com/users/simonbrunel/followers","following_url":"https://api.github.com/users/simonbrunel/following{/other_user}","gists_url":"https://api.github.com/users/simonbrunel/gists{/gist_id}","starred_url":"https://api.github.com/users/simonbrunel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simonbrunel/subscriptions","organizations_url":"https://api.github.com/users/simonbrunel/orgs","repos_url":"https://api.github.com/users/simonbrunel/repos","events_url":"https://api.github.com/users/simonbrunel/events{/privacy}","received_events_url":"https://api.github.com/users/simonbrunel/received_events","type":"User","site_admin":false},"open_issues":28,"closed_issues":2,"state":"open","created_at":"2018-09-16T09:41:05Z","updated_at":"2018-10-18T19:28:50Z","due_on":null,"closed_at":null},"comments":28,"created_at":"2018-09-20T13:47:48Z","updated_at":"2018-10-18T20:21:39Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/chartjs/Chart.js/pulls/5741","html_url":"https://github.com/chartjs/Chart.js/pull/5741","diff_url":"https://github.com/chartjs/Chart.js/pull/5741.diff","patch_url":"https://github.com/chartjs/Chart.js/pull/5741.patch"},"body":"~This patch introduces the `minSize` dataset property for bar charts.~ \r\n\r\nThis patch introduces the `minBarLength` scale property for bar charts. The value, if set, defines the minimum size a bar should be rendered when the value would be too low for the bar to appear visible.\r\n\r\nWorks for both vertical and horizontal bars.\r\n\r\nFixes #2959 \r\nFixes #3915\r\n\r\n~Live example on jsbin: https://jsbin.com/rodurop/edit?html,js,output~\r\n\r\n\r\n> Edit(SB): \"Fixes #xxxx\" to close associated tickets "},"comment":{"url":"https://api.github.com/repos/chartjs/Chart.js/issues/comments/431148568","html_url":"https://github.com/chartjs/Chart.js/pull/5741#issuecomment-431148568","issue_url":"https://api.github.com/repos/chartjs/Chart.js/issues/5741","id":431148568,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMTE0ODU2OA==","user":{"login":"ahonen89","id":32383149,"node_id":"MDQ6VXNlcjMyMzgzMTQ5","avatar_url":"https://avatars2.githubusercontent.com/u/32383149?v=4","gravatar_id":"","url":"https://api.github.com/users/ahonen89","html_url":"https://github.com/ahonen89","followers_url":"https://api.github.com/users/ahonen89/followers","following_url":"https://api.github.com/users/ahonen89/following{/other_user}","gists_url":"https://api.github.com/users/ahonen89/gists{/gist_id}","starred_url":"https://api.github.com/users/ahonen89/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahonen89/subscriptions","organizations_url":"https://api.github.com/users/ahonen89/orgs","repos_url":"https://api.github.com/users/ahonen89/repos","events_url":"https://api.github.com/users/ahonen89/events{/privacy}","received_events_url":"https://api.github.com/users/ahonen89/received_events","type":"User","site_admin":false},"created_at":"2018-10-18T20:21:39Z","updated_at":"2018-10-18T20:21:39Z","author_association":"NONE","body":"> \r\n> \r\n> We could make `minBarLength` [scriptable](http://www.chartjs.org/docs/latest/general/options.html#scriptable-options):\r\n> \r\n> ```js\r\n> minBarLength: function(context) {\r\n>    // 5px bar length for non zero values, else 0\r\n>    return context.dataset.data[context.dataIndex] ? 5 : 0\r\n> }\r\n> ```\r\n> \r\n> Though, that would require much more effort / changes, so I would not implement it now. We can still merge and add the scriptable part in another PR (would not be a breaking change).\r\n> \r\n> What do you think? Any other idea?\r\n\r\nActually this is a great idea, allowing a lot of flexibility."}}
{"actor":{"display_login":"ahonen89"}}