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

migrate to package:web and implement keyboard detection #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

raldhafiri
Copy link

Add support for notifying soft keyboard visibility on web and migrate to package:web

@raldhafiri raldhafiri force-pushed the master branch 4 times, most recently from 4f5e4da to ad25e70 Compare April 9, 2024 01:52
@raldhafiri
Copy link
Author

Hi @MisterJimson .. could you please review this pull request

@ot-repo
Copy link

ot-repo commented May 15, 2024

Hello @MisterJimson I am very interested in WASM and appreciate it if you can review the pull request and merge it.
Thank you!

@raldhafiri
Copy link
Author

Hello @MisterJimson I am very interested in WASM and appreciate it if you can review the pull request and merge it. Thank you!

I would recommend you to create pubspec_overrides.yaml and use it until it get merged or fork my repo
I know its not the best solution but hopefully it will be merged soon to support WASM

dependency_overrides:
  flutter_keyboard_visibility_web:
    git:
      url: https://github.com/raldhafiri/flutter_keyboard_visibility.git
      ref: master
      path: flutter_keyboard_visibility_web

@ot-repo
Copy link

ot-repo commented May 15, 2024

Hello @MisterJimson I am very interested in WASM and appreciate it if you can review the pull request and merge it. Thank you!

I would recommend you to create pubspec_overrides.yaml and use it until it get merged or fork my repo I know its not the best solution but hopefully it will be merged soon to support WASM

dependency_overrides:
  flutter_keyboard_visibility_web:
    git:
      url: https://github.com/raldhafiri/flutter_keyboard_visibility.git
      ref: master
      path: flutter_keyboard_visibility_web

Thank you for the suggestion. At least we can test our solution now. Hope it gets merged as soon as possible.
Regards

@dxvid-pts
Copy link

@MisterJimson This is blocking WASM compilation for our project (see: https://docs.flutter.dev/platform-integration/web/wasm)

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.

None yet

3 participants