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

[web] Make HtmlElementView more useful #126176

Closed
wants to merge 1 commit into from

Conversation

mdebbar
Copy link
Contributor

@mdebbar mdebbar commented May 5, 2023

Add the ability to set html attributes and styles on the HtmlElementView widget. This makes the widget much more convenient to use and avoids the registration of multiple view factories unnecessarily (e.g. pointer_interceptor).

@flutter-dashboard flutter-dashboard bot added the framework flutter/packages/flutter repository. See also f: labels. label May 5, 2023
@goderbauer
Copy link
Member

(triage) @mdebbar Do you still have plans for this draft?

@ditman
Copy link
Member

ditman commented Jun 2, 2023

@goderbauer this was only a draft to discuss options, I think @mdebbar is actually going to land something different soon (should we fully close this PR so it doesn't uses resources during triage?)

@mdebbar
Copy link
Contributor Author

mdebbar commented Jun 2, 2023

Yeah, we decided on a different plan here.

@mdebbar mdebbar closed this Jun 2, 2023
auto-submit bot pushed a commit to flutter/engine that referenced this pull request Jun 2, 2023
In order to make `HtmlElementView` more useful, we need to expose an API for getting the created platform view element. ~~This would allow `HtmlElementView` to set attributes and styles on the element.~~

This would allow us to add a new callback to the `HtmlElementView` that would return the result of calling the registered platformViewFactory to the user, after it's been created and added to the DOM, so the user can do with it as they please.

Required by flutter/flutter#126176
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants