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

[BUG] 'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent' #441

Closed
zs-dima opened this issue Jul 8, 2021 · 12 comments
Labels
awaiting-further-comments bug Something isn't working

Comments

@zs-dima
Copy link

zs-dima commented Jul 8, 2021

Flutter 2.4.0-0.0.pre • channel dev
Framework • revision 96bbcd006f (8 days ago) • 2021-06-30 03:43:41 -0400
Engine • revision c633b2a6a0
Tools • Dart 2.14.0 (build 2.14.0-258.0.dev)

Packages installed
native_pdf_view

Error:

photo_view-0.11.1/lib/src/core/photo_view_gesture_detector.dart:106:29: Error: The argument type 'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent'.
 - 'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('/C:/dev/flutter/packages/flutter/lib/src/gestures/events.dart').
 - 'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('/C:/dev/flutter/packages/flutter/lib/src/gestures/events.dart').
    super.addAllowedPointer(event);
@zs-dima zs-dima added the bug Something isn't working label Jul 8, 2021
@radvansky-tomas
Copy link

@zs-dima change channel do STABLE

@zs-dima
Copy link
Author

zs-dima commented Jul 11, 2021

@radvansky-tomas thanks for reply.
I am using Dev channel for this project. With Stable channel this project could not be compiled.

@radvansky-tomas
Copy link

I understand, but you cannot expect plugins to move to DEV, fork it...fix that line and maybe send PR back so its prepared when it makes to stable version

@zs-dima
Copy link
Author

zs-dima commented Jul 11, 2021

@radvansky-tomas but all plugin working well with Dev channel except this one, that quite unusual
I expected Dev channel could have some bugs and recent fixes, but bugs fixes quite fast usually.

@mayder
Copy link

mayder commented Jul 12, 2021

https://flutter.dev/docs/release/breaking-changes/gesture-recognizer-add-allowed-pointer

@Vinzent03
Copy link

Does the fix work on stable too? If so, I would love to see the pr merged, because the breaking change is on beta channel already too.

@SDAChess
Copy link

SDAChess commented Jul 15, 2021

Does the fix work on stable too? If so, I would love to see the pr merged, because the breaking change is on beta channel already too.

I tested my fix on stable and it works. I don't think the PR is going to get merged any time soon, because I'm not sure how much often the author releases a new version. Still if you need the fix you can take my branch as reference for the library code with something like this in your pubspec :

photo_view:
       git:
         url: git@github.com:SDAChess/photo_view.git
         ref: SDAChess-patch-441

Not quite sure it would work though but still a good thing to try.

@shengdoushi
Copy link

Does the fix work on stable too? If so, I would love to see the pr merged, because the breaking change is on beta channel already too.

I tested my fix on stable and it works. I don't think the PR is going to get merged any time soon, because I'm not sure how much often the author releases a new version. Still if you need the fix you can take my branch as reference for the library code with something like this in your pubspec :

photo_view:
       git:
         url: git@github.com:SDAChess/photo_view.git
         ref: SDAChess-patch-441

Not quite sure it would work though but still a good thing to try.

Some library using photo_view's some version, how to force use git's version?

@jseminck
Copy link

@renancaraujo, sorry for pinging you but is there any chance we can get the PR merged? The latest flutter channel branch contains much needed performance improvements (shader compilations) and we have some dependencies that rely on this package. Thanks so much for your work!

@renancaraujo
Copy link
Member

Hello people,

I just woke up from a coma and created a PR that addresses this problem.
#445 should work for stable, beta, dev, and master channels.

I will do some testing and it should be included in the next release.

renancaraujo added a commit that referenced this issue Jul 19, 2021
* Fix(#441) add allowed pointer

* shame on me
@renancaraujo
Copy link
Member

This should be fixed on 0.12.0: https://github.com/fireslime/photo_view/releases/tag/0.12.0

@no-response
Copy link

no-response bot commented Jul 26, 2021

Since we got no answer for a while, I am automatically closing this issue for the sake of organization. If there is any news on that, feel free to retake the discussion.

@no-response no-response bot closed this as completed Jul 26, 2021
@bluefireteam bluefireteam locked and limited conversation to collaborators Jul 27, 2021
@bluefireteam bluefireteam unlocked this conversation Jul 27, 2021
SergeShkurko pushed a commit to ScerIO/packages.flutter that referenced this issue Aug 3, 2021
* Update photo_view dependency to 0.12.0

An issue was fixed in the `photo_view` dependency that made it impossible to use with the newest flutter version (2.3, currently in beta). Bumping the dependency here as well.

See: bluefireteam/photo_view#441

* Update pubspec.lock
@bluefireteam bluefireteam locked and limited conversation to collaborators Sep 16, 2021
@bluefireteam bluefireteam deleted a comment from flutterq Sep 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
awaiting-further-comments bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants