Skip to content

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)

@raldhafiri
Copy link
Author

Any chances this will get merge it has been 5 months since pull request opened?
much apperciated everyone

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.

6 participants