-
-
Notifications
You must be signed in to change notification settings - Fork 48
feat: add key event #55
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
base: master
Are you sure you want to change the base?
Conversation
|
Right now the change is on my fork, we can fork the repo into our org if you don't want to accept the patches that we need to support appflowy-editor. I'm mainly I'm using this PR to establish some communication with you about generating these bridge libraries because hand-writing them is a lot of work. Any updates on that? |
| sdk: flutter | ||
| dart_eval: ^0.6.0 | ||
| dart_eval: | ||
| path: ../dart_eval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this change
| description: Flutter bridge library for dart_eval, enabling creation of fully dynamic Flutter apps and widgets that can be loaded from a file or the Internet at runtime. | ||
| version: 0.6.0 | ||
| homepage: https://github.com/ethanblake4/flutter_eval | ||
| publish_to: "none" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this change as well.
| $CustomPainter.wrap(this.$value); | ||
| } | ||
|
|
||
| class $CustomPainter$bridge extends CustomPainter with $Bridge<CustomPainter> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be completed
| import 'package:flutter/material.dart'; | ||
| import 'package:flutter/rendering.dart'; | ||
|
|
||
| class $CustomPainter implements $Instance { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove wrapper class if using bridge
| } | ||
| } | ||
|
|
||
| class $CustomPaint$bridge extends CustomPaint with $Bridge<CustomPaint> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CustomPainter should be a bridge class, but CustomPaint should be a wrapper
|
Thanks for this work @a-wallen ! I'm happy to accept it into flutter_eval (assuming the review comments are addressed). As far as binding generation, I don't have any immediate updates to share, but your feedback is noted. I can tell you it will not be completed in the next few months (barring a major change like this project getting sponsored), but is certainly something I want to address. I assume you already know this, but ChatGPT and GitHub Copilot do a reasonably good job if prompted with a completed example of a binding class definition. |
I'm working on a bridge library for appflowy-editor so that we can dynamically load developer plugins in our rich text editor. We need
KeyEventResultand a couple other classes.