Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Jan 14, 2021

Description

Following up on #23408. Fixes addPlaceholder JS bindings.

Related Issues

Fixes flutter/flutter#73278

Tests

Added a new test to canvaskit_api_test.dart.

@vsmenon
Copy link
Contributor

vsmenon commented Jan 14, 2021

lgtm - thanks!

@chinmaygarde chinmaygarde added the platform-web Code specifically for the web engine label Jan 14, 2021
@yjbanov yjbanov added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Jan 20, 2021
@fluttergithubbot fluttergithubbot merged commit 688578e into flutter:master Jan 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 20, 2021
hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes platform-web Code specifically for the web engine waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[web] CanvasKit error on ParagraphBuilder.addPlaceholder

6 participants