Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detach attach #8

Closed
wants to merge 23 commits into from
Closed

Detach attach #8

wants to merge 23 commits into from

Conversation

LovelyAndy
Copy link

No description provided.

oscard0m and others added 23 commits October 16, 2021 22:18
…s#9757)

* ci(workflow): add 'npm' cache for actions/setup-node in .github/workflows

* style(workflows): restore single quotes and remove unnecessary 'null'
* Fix: Typo 'is less' -> 'are lesser'

* Update plugin.decimation.js
Inserted a hyperlink on chart.js logo 
So that users can easily be redirected to the HOME PAGE of the web application
* Fix cleaning up metasets

I believe it's a mistake to only delete the metaset if it has a valid controller; see f191f2f for where this behavior was introduced.

This is a minimal fix for chartjs#9653; as discussed there, it may also be worth updating `updateHoverStyle`.

As of chartjs#7030, `this._metasets` should always be defined, so checking whether it's undefined is no longer necessary.

* Add a test covering metaset behavior

* Add a regression test for chartjs#9653; fix `toHaveSize` usage

* Fix test failure
* make object available for bar as border Radius

* override only the bar borderWidth prop

* spacing
* Remove scriptability from defaults.font types

* Remove failing test
* Define with let to avoid "assignment to constant" errors

Thanks for this example. Defining `label` with `const` rather than `let` results in `Uncaught TypeError: Assignment to constant variable.`

* Another case where const needs to be replaced with let.

* Requested cases where const needs to be replaced with let +1 (style).
* update type of scale position in docs, add explanation sentence to object position

* improve clarification

* grammar/spelling
* Fix type linting and indent errors
* Properly indent comments too
* add autopadding to types

* fix failing test
@LovelyAndy LovelyAndy closed this Nov 23, 2021
@LovelyAndy LovelyAndy deleted the detach-attach branch November 23, 2021 12:57
@LovelyAndy LovelyAndy restored the detach-attach branch November 23, 2021 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.