Skip to content

Add missing modifierKey properties to Options#502

Merged
kurkle merged 2 commits intochartjs:masterfrom
jbelien:patch-1
May 7, 2021
Merged

Add missing modifierKey properties to Options#502
kurkle merged 2 commits intochartjs:masterfrom
jbelien:patch-1

Conversation

@jbelien
Copy link
Copy Markdown
Contributor

@jbelien jbelien commented May 7, 2021

  • Missing modifierKey to PanOptions
  • Missing wheelModifierKey to ZoomOptions

- Missing `modifierKey` to `PanOptions`
- Missing `wheelModifierKey` to `ZoomOptions`
Copy link
Copy Markdown
Member

@kurkle kurkle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[test-lint] [lint-types] /home/runner/work/chartjs-plugin-zoom/chartjs-plugin-zoom/types/options.d.ts
[test-lint] [lint-types] 51:1 error Trailing spaces not allowed no-trailing-spaces

Apply lint fix (remove trailing spaces).
@kurkle kurkle merged commit e0973e0 into chartjs:master May 7, 2021
@kurkle
Copy link
Copy Markdown
Member

kurkle commented May 7, 2021

Thanks @jbelien

@jbelien jbelien deleted the patch-1 branch May 7, 2021 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants