Skip to content

[file_selector_web] migrate to pkg:web - #5413

Merged
auto-submit[bot] merged 25 commits into
mainfrom
file_selector_web
Nov 17, 2023
Merged

[file_selector_web] migrate to pkg:web#5413
auto-submit[bot] merged 25 commits into
mainfrom
file_selector_web

Conversation

@kevmoo

@kevmookevmoo commented Nov 16, 2023

Copy link
Copy Markdown
Contributor

This allows this package to be used in a web app compiled to Wasm.

Helps unblock flutter/devtools#6606

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@kevmoo
kevmoo requested a review from ditman as a code ownerNovember 16, 2023 19:25
@kevmookevmoo changed the title file selector web[file_selector_web] migrate to pkg:webNov 16, 2023

@ditmanditman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, and CI seems happy, let's go!

final HTMLInputElement inputElement =
input ?? (createElementTag('input') as HTMLInputElement)
..type = 'file';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be nice if this asserted that input.type == 'file', to retain some of the old type safety.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not worried about this, since it's text only

@@ -1,3 +1,7 @@
## 0.9.3

* Updates minimum supported SDK version to Dart 3.2.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
*Updates minimum supported SDK version to Dart 3.2.
*Migrates web implementation to `package:web` from `dart:html`.
* Updates minimum supported SDK version to Dart 3.2.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤷 – for the most part, this is not a user visible change so I omitted that bit.

@kevmookevmoo added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 17, 2023
@auto-submit
auto-submitBot merged commit 42dbb75 into mainNov 17, 2023
@auto-submit
auto-submitBot deleted the file_selector_web branch November 17, 2023 00:40
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 20, 2023
auto-submitBot pushed a commit to flutter/flutter that referenced this pull request Nov 20, 2023
flutter/packages@07b4b29...c5443ad
2023-11-20 JeroenWeener@users.noreply.github.com [webview_flutter] Support for handling basic authentication requests (Platform Interface) (flutter/packages#5362)
2023-11-18 engine-flutter-autoroll@skia.org Roll Flutter from 53a57ad to 6cf9ab0 (31 revisions) (flutter/packages#5426)
2023-11-18 kevmoo@users.noreply.github.com [shared_preferences_web] migrate to pkg:web (flutter/packages#5414)
2023-11-18 stuartmorgan@google.com [ci] Roll minimum allowable Flutter to 3.10 (flutter/packages#5425)
2023-11-18 43054281+camsim99@users.noreply.github.com [path_provider_android] Run tests on AVDs running Android 34 (flutter/packages#5222)
2023-11-17 tarrinneal@gmail.com [pigeon] isEnum, isClass, fix swift casting, default values, optional method arguments, named method arguments (flutter/packages#5355)
2023-11-17 stuartmorgan@google.com [plugin_platform_interface] Switch mixin to `mixin class` (flutter/packages#5420)
2023-11-17 stuartmorgan@google.com [go_router] Fixes use of `Iterable` (flutter/packages#5421)
2023-11-17 stuartmorgan@google.com [pigeon] Adds `analyzer` 6.x compatibility (flutter/packages#5418)
2023-11-17 stuartmorgan@google.com Update release step to 3.16 (flutter/packages#5416)
2023-11-17 kevmoo@users.noreply.github.com [file_selector_web] migrate to pkg:web (flutter/packages#5413)
2023-11-16 stuartmorgan@google.com [video_player] Improve macOS frame management (flutter/packages#5078)
2023-11-16 engine-flutter-autoroll@skia.org Roll Flutter from e8c2bb1 to 53a57ad (39 revisions) (flutter/packages#5412)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@reidbakerreidbaker mentioned this pull request Jan 12, 2024
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
This allows this package to be used in a web app compiled to Wasm.
Helps unblock flutter/devtools#6606
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
This allows this package to be used in a web app compiled to Wasm.
Helps unblock flutter/devtools#6606
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appp: file_selectorplatform-web

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@kevmoo@ditman