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

fix: SizeValue interface ts error #58

Closed
wants to merge 1 commit into from
Closed

fix: SizeValue interface ts error #58

wants to merge 1 commit into from

Conversation

AlexSND
Copy link

@AlexSND AlexSND commented Oct 18, 2021

Fix error TS2412: Property '2' of type '"px" | "%" | undefined' is not assignable to numeric index type 'string | number'.

@AlexSND AlexSND closed this Oct 29, 2021
@AlexSND AlexSND deleted the types-fix branch October 29, 2021 10:43
@metal3d
Copy link

metal3d commented Oct 12, 2022

Hi, I still have the issue with Angular. Could you please merge the fix and create a new release ?

@metal3d
Copy link

metal3d commented Oct 12, 2022

Actually, the fix is not implemented in index.d.ts - could you please apply the fix ?

With Angular, the error is fatal.

metal3d added a commit to metal3d/Croppr.js that referenced this pull request Oct 12, 2022
The pull jamesssooi#58 is not merged and this yields an error using Croppr in
Angular.
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.

2 participants