-
Notifications
You must be signed in to change notification settings - Fork 145
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
Compatibility with Dart 2.10. #217
base: master
Are you sure you want to change the base?
Conversation
This fixes #215. |
I would also be happy if this is merged. I think screenshots would be a valuable tool, but the unresponsiveness worries me a bit. |
Is anyone willing to publish a working fork of screenshots if there is no activity soon? I think this is a great project and it would help a lot of developers. |
Ping @mmcc007 |
Necessary to replace the 'resources' package.
Fixing many small bugs, lots of null pointers and badly converted code.
Don't hang forever on devices where we're unable to change the locale.
@teolemon I've done some work on trying to get it up to date with the latest dart/flutter releases but it is a fair bit of work. Any help and testing would be appreciated. I've worked out a good number of bugs, but I'm sure there are more remaining. All my work is done on my fork for now. |
Sounds good, but this seems to be a very essential plugin, depending on how much time you @trygvis want to invest here you could also ask fluttercommunity if you could transfer your repository over there. Just as an idea |
Found this thread here about bringing it up to date. Can I get an update on when this could be useable? |
Any other package to do the same? |
This depends on mmcc007/tool_base#2, mmcc007/tool_mobile#1 and mmcc007/fake_process_manager#1.
These changes has been done as a part of getting screenshots to work with Dart 2.10. The new screenshots work in the same way as earlier builds for my app.