]> git.ipfire.org Git - thirdparty/Chart.js.git/commitdiff
Remove unused bar property and confusing docs referring to it (#7333)
authorBen McCann <322311+benmccann@users.noreply.github.com>
Tue, 12 May 2020 20:42:48 +0000 (13:42 -0700)
committerGitHub <noreply@github.com>
Tue, 12 May 2020 20:42:48 +0000 (16:42 -0400)
docs/docs/developers/charts.md
docs/docs/getting-started/v3-migration.md
src/controllers/controller.bar.js

index 3832232861b99353675ea1160e2b3f2e7aae2f80..75b3a1c4a7a44f8aad9fe8bb38f20ec6a396787f 100644 (file)
@@ -114,6 +114,3 @@ new Chart(ctx, {
     options: options
 });
 ```
-
-### Bar Controller
-The bar controller has a special property that you should be aware of. To correctly calculate the width of a bar, the controller must determine the number of datasets that map to bars. To do this, the bar controller attaches a property `bar` to the dataset during initialization. If you are creating a replacement or updated bar controller, you should do the same. This will ensure that charts with regular bars and your new derived bars will work seamlessly.
index 653938e6ed96a5be5981720f1686c862d4da8f8e..c9f573112e039d5a9c56a7691a0a0379a457e4fa 100644 (file)
@@ -197,8 +197,9 @@ The following properties and methods were removed:
 * `Chart.Tooltip` is now provided by the tooltip plugin. The positioners can be accessed from `tooltipPlugin.positioners`
 * `ILayoutItem.minSize`
 
-#### Dataset Controller
+#### Dataset Controllers
 
+* `BarController.getDatasetMeta().bar`
 * `DatasetController.addElementAndReset`
 * `DatasetController.createMetaData`
 * `DatasetController.createMetaDataset`
index fa0806ed01e1b4538448842cf3196f45ecd5df3a..0b21e96b3a43a4ea75d86bb2b3611d87c46ca75a 100644 (file)
@@ -246,7 +246,6 @@ export default class BarController extends DatasetController {
 
                const meta = me._cachedMeta;
                meta.stack = me.getDataset().stack;
-               meta.bar = true;
        }
 
        update(mode) {