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

Conversation

@mdebbar
Copy link
Contributor

@mdebbar mdebbar commented Dec 18, 2020

Description

This PR turns on the new rich text implementation by default.

It's still possible to opt-out by passing false to the flag:

flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_EXPERIMENTAL_CANVAS_RICH_TEXT=false

Related Issues

Fixes flutter/flutter#55587
Fixes flutter/flutter#39226
Fixes flutter/flutter#61992
Fixes flutter/flutter#63638
Fixes flutter/flutter#66089
Fixes flutter/flutter#71176

@mdebbar mdebbar added the platform-web Code specifically for the web engine label Dec 18, 2020
@mdebbar mdebbar requested a review from ferhatb December 18, 2020 00:36
@mdebbar mdebbar changed the title [DO NOTE MERGE][web] Enable the new rich paragraph implementation [web] Enable the new rich paragraph implementation Jan 8, 2021
@mdebbar mdebbar force-pushed the rich_enable branch 3 times, most recently from 5c0f99c to ef5e678 Compare January 11, 2021 23:46
@mdebbar mdebbar 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 12, 2021
@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Linux Web Engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@fluttergithubbot fluttergithubbot removed 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 12, 2021
@mdebbar mdebbar merged commit 51dd6aa into flutter:master Jan 12, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 12, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 12, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
zanderso added a commit that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 13, 2021
zanderso pushed a commit to flutter/flutter that referenced this pull request Jan 13, 2021
* 51dd6aa [web] Enable the new rich paragraph implementation (flutter/engine#23162)

* 0f9fc3d Roll Skia from 7cf3addb1bd8 to 93c2d81f199a (1 revision) (flutter/engine#23614)

* a1e7424 [dart-runner] Avoid calling Destroy on nullptr (flutter/engine#23608)

* 846d958 Windows textures: Add placeholder flutter_texture_registrar.h (flutter/engine#23623)

* 55afc18 Roll Dart SDK from 7fcbd388b620 to ef8bf7f0a667 (5 revisions) (flutter/engine#23628)

* d2b8154 [canvaskit] apply invser scale on the left (flutter/engine#23550)

* be1a3c0 Roll Fuchsia Linux SDK from UB6RsTbdU... to FfWbbB4r8... (flutter/engine#23633)

* f743c89 Roll Skia from 93c2d81f199a to 9fd75e96d712 (29 revisions) (flutter/engine#23635)

* cf42dbe Roll wuffs to google/wuffs@c86add2 (flutter/engine#23607)

* f1278d0 Link SkShaper/SkParagraph into the engine by default (flutter/engine#23626)

* fb56b4b Android deeplink sends "path + query" instead of just path (flutter/engine#23561)

* 22bb891 Plumbing refactor to allow the usage of Dart_CreateIsolateInGroup (flutter/engine#23549)

* 20991a5 Add accessibility suport to Linux shell. (flutter/engine#19634)

* 145922b Roll Dart SDK from ef8bf7f0a667 to 636ff0ec97e0 (1 revision) (flutter/engine#23639)

* 176ae6e Roll Fuchsia Mac SDK from oll0Dgp9o... to JSzm8D59u... (flutter/engine#23641)

* d2320a8 Roll Dart SDK from 636ff0ec97e0 to d3d7b77e8165 (1 revision) (flutter/engine#23642)

* 8bdd099 Roll Skia from 9fd75e96d712 to 38ca513408d1 (1 revision) (flutter/engine#23643)

* cc572e1 Roll Skia from 38ca513408d1 to be2a8614c5d6 (2 revisions) (flutter/engine#23644)

* e9383a0 Roll Dart SDK from d3d7b77e8165 to 010633edc631 (1 revision) (flutter/engine#23645)

* 9d6ed8b Roll Fuchsia Linux SDK from FfWbbB4r8... to BUsKF6z4t... (flutter/engine#23646)

* 6d55dd4 Roll Dart SDK from 010633edc631 to 724d9e5e7d71 (1 revision) (flutter/engine#23647)

* 299f081 Roll Skia from be2a8614c5d6 to 0d7de6bc9ac3 (1 revision) (flutter/engine#23648)

* 0ec99cf Roll Fuchsia Mac SDK from JSzm8D59u... to BsUY1yjWh... (flutter/engine#23650)

* 859494f Revert "[web] Enable the new rich paragraph implementation (#23162)" (flutter/engine#23651)

* 648dae9 Roll Skia from 0d7de6bc9ac3 to 92969f265686 (7 revisions) (flutter/engine#23652)
mdebbar added a commit that referenced this pull request Jan 14, 2021
mdebbar added a commit that referenced this pull request Jan 14, 2021
gspencergoog pushed a commit to gspencergoog/engine that referenced this pull request Jan 20, 2021
gspencergoog pushed a commit to gspencergoog/engine that referenced this pull request Jan 20, 2021
gspencergoog pushed a commit to gspencergoog/engine that referenced this pull request Jan 20, 2021
@dannnnthemannnn
Copy link

Hi @mdebbar, is this functionality in beta yet? I am happy to pass the flag manually but my first test run with the new flag didnt seem to allow highlighting of multiline text.

@lukemadera
Copy link

@mdebbar Is this merged in to a main Flutter version? I'm having multiline issues as noted here: https://github.com/flutter/flutter_markdown/issues/315#issuecomment-773264097

This issue you closed, but it does not seem to actually be fixed yet? Or am I missing something? flutter/flutter#63638

@mdebbar
Copy link
Contributor Author

mdebbar commented Feb 4, 2021

@dannnnthemannnn @lukemadera this is available on flutter master or dev channels. It didn't make it to beta or stable yet.

If you could try the master channel, that would be great! Please let me know if you still see any problems. File an issue and mention me on it.

@lukemadera
Copy link

Thanks for the prompt reply! My use case is through markdown so I think I'll need to wait for it to get on stable - any idea when that will be?

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

Projects

None yet

6 participants