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

Inkwell long press is not working on iOS devices while flutter add as a module. #148661

Open
balasubramaniam-paisa opened this issue May 20, 2024 · 15 comments
Labels
a: existing-apps Integration with existing apps via the add-to-app flow f: gestures flutter/packages/flutter/gestures repository. found in release: 3.22 Found to occur in 3.22 found in release: 3.23 Found to occur in 3.23 framework flutter/packages/flutter repository. See also f: labels. has reproducible steps The issue has been confirmed reproducible and is ready to work on P2 Important issues not at the top of the work list platform-ios iOS applications specifically team-ios Owned by iOS platform team triaged-ios Triaged by iOS platform team

Comments

@balasubramaniam-paisa
Copy link

Steps to reproduce

Create a native iOS app, then integrate a module from a Flutter project. Add an InkWell widget with properties for both onTap and long press. While onTap is functioning properly, long press isn't responsive unless two fingers are pressed before initiating the long press.

Expected results

Both long press and onTap need to work fine.

Actual results

Long press is not working, but onTap is working fine.

Code sample

Code sample
Column(
          children: <Widget>[
            InkWell(
              onLongPress: () {
                 // this event not working
              },
              onTap: () {
                // this event works fine
              },
              child: const Text('click me'),
            ),
            const AppDivider(),
          ],
        )

Screenshots or Video

Screenshots / Video demonstration

https://youtube.com/shorts/fsISyUNkObk?feature=share

Logs

Logs
[Paste your logs here]

Flutter Doctor output

Doctor output
[Paste your output here]
@balasubramaniam-paisa balasubramaniam-paisa changed the title Inkwell long press is not working on some iOS devices. Inkwell long press is not working on iOS devices while flutter add as a module. May 20, 2024
@darshankawar darshankawar added the in triage Presently being triaged by the triage team label May 20, 2024
@darshankawar
Copy link
Member

@balasubramaniam-paisa
Can you provide us a sample repo that we can clone directly and run to verify this further ?
Also provide your flutter doctor -v.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 20, 2024
@balasubramaniam-paisa
Copy link
Author

After create flutter module for IOS, inkwell long press not working, below code we used to create IOS flutter module
flutter create --template module my_flutter

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 20, 2024
@darshankawar
Copy link
Member

@balasubramaniam-paisa
Please don't provide half hearted details. I created the template as mentioned and used the code snippet you provided earlier, but not sure what AppDivider() is. That's why we request you to provide us complete code sample or a repo that we can clone in order to avoid this back and forth for details.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 21, 2024
@balasubramaniam-paisa
Copy link
Author

@darshankawar Repo added kindly check and update
https://github.com/balasubramaniam-paisa/InkwellDetect

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 21, 2024
@darshankawar
Copy link
Member

@balasubramaniam-paisa Check if this resembles your case or not.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 22, 2024
@balasubramaniam-paisa

This comment was marked as outdated.

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 22, 2024
@darshankawar

This comment was marked as outdated.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 23, 2024
@balasubramaniam-paisa
Copy link
Author

@darshankawar The Flutter module click action is not working in the iOS native child view, as you suggested. However, it does not exactly match our scenario. Could you please review our scenario and provide a solution?

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 23, 2024
@darshankawar
Copy link
Member

Thanks for the update. Using the repo provided, I couldn't replicate the issue. Can you provide what flutter version and iOS device this is occuring on ?

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 24, 2024
@balasubramaniam-paisa
Copy link
Author

@darshankawar Did you run the iOS zip file? Please check again with the iOS setup. All devices are experiencing this issue.

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 24, 2024
@balasubramaniam-paisa

This comment was marked as off-topic.

@darshankawar
Copy link
Member

Yes, tried again per code given but was unable to replicate it, as shown below:

148661.mov

Tried using latest master version.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 27, 2024
@balasubramaniam-paisa
Copy link
Author

@darshankawar You are directly running the Flutter project. Don't run it directly; instead, try running Xcode. Unzip the iOS native app code and try running it in Xcode.

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-05-27.at.12.23.22.mp4

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 27, 2024
@darshankawar
Copy link
Member

Thanks for the update. I tried again and was able to replicate this as reported.

stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.22.0, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.22.0 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5dcb86f68f (5 days ago), 2024-05-09 07:39:20 -0500
    • Engine revision f6344b75dc
    • Dart version 3.4.0
    • DevTools version 2.34.3
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.23.0-8.0.pre.8, on macOS 12.2.1 21D62 darwin-x64,
    locale en-GB)
    • Flutter version 3.23.0-8.0.pre.8 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8fd8e66de4 (4 hours ago), 2024-05-26 21:49:30 -0400
    • Engine revision 469193ef0e
    • Dart version 3.5.0 (build 3.5.0-191.0.dev)
    • DevTools version 2.36.0-dev.10
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.
      
[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.



@darshankawar darshankawar added platform-ios iOS applications specifically framework flutter/packages/flutter repository. See also f: labels. labels May 27, 2024
@darshankawar darshankawar added f: gestures flutter/packages/flutter/gestures repository. a: existing-apps Integration with existing apps via the add-to-app flow has reproducible steps The issue has been confirmed reproducible and is ready to work on found in release: 3.22 Found to occur in 3.22 found in release: 3.23 Found to occur in 3.23 team-ios Owned by iOS platform team and removed in triage Presently being triaged by the triage team labels May 27, 2024
@loic-sharma loic-sharma added P2 Important issues not at the top of the work list triaged-ios Triaged by iOS platform team fyi-framework For the attention of Framework team labels May 29, 2024
@goderbauer goderbauer added fyi-design For the attention of Design Languages team triaged-framework Triaged by Framework team labels Jun 4, 2024
@flutter-triage-bot flutter-triage-bot bot removed fyi-framework For the attention of Framework team triaged-framework Triaged by Framework team labels Jun 4, 2024
@Piinks Piinks added the triaged-design Triaged by Design Languages team label Jun 5, 2024
@flutter-triage-bot flutter-triage-bot bot removed fyi-design For the attention of Design Languages team triaged-design Triaged by Design Languages team labels Jun 5, 2024
@balasubramaniam-paisa
Copy link
Author

@darshankawar Any update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: existing-apps Integration with existing apps via the add-to-app flow f: gestures flutter/packages/flutter/gestures repository. found in release: 3.22 Found to occur in 3.22 found in release: 3.23 Found to occur in 3.23 framework flutter/packages/flutter repository. See also f: labels. has reproducible steps The issue has been confirmed reproducible and is ready to work on P2 Important issues not at the top of the work list platform-ios iOS applications specifically team-ios Owned by iOS platform team triaged-ios Triaged by iOS platform team
Projects
None yet
Development

No branches or pull requests

5 participants