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

Conversation

@jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams commented Oct 20, 2022

Next step in go/impeller-geometry, removes specialized stroke contents and allows strokes to use color sources

Screen Shot 2022-10-20 at 8 59 04 AM

flutter/flutter#104757

@jonahwilliams jonahwilliams requested a review from bdero October 20, 2022 16:02
@jonahwilliams
Copy link
Contributor Author

This doesn't cover vertices yet, which need a bit more plumbing (though that will help with SVG so I'm going to take a look later).

But the generic position_no_color vertex shader is missing a matrix uniform. I don't know if that means we need another variant for gradients or if I misread the document

@jonahwilliams
Copy link
Contributor Author

Updated drawVertices to support color sources. Doesn't handle blending with per-vertex color, et cetera

@jonahwilliams jonahwilliams changed the title [Impeller] remove solid stroke contents and allow strokes to use color sources [Impeller] remove solid stroke contents and allow strokes/vertices to use color sources Oct 20, 2022
@jonahwilliams jonahwilliams self-assigned this Oct 20, 2022
Copy link
Member

@bdero bdero left a comment

Choose a reason for hiding this comment

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

LGTM!

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 20, 2022
@auto-submit auto-submit bot merged commit b923707 into flutter:main Oct 20, 2022
@jonahwilliams jonahwilliams deleted the remove_stroke_contents branch October 20, 2022 21:10
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 20, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 21, 2022
zanderso pushed a commit to flutter/flutter that referenced this pull request Oct 21, 2022
* e27e196 Add touch-input-test to test_suites.yaml (flutter/engine#36900)

* b923707 [Impeller] remove solid stroke contents and allow strokes/vertices to use color sources (flutter/engine#36896)

* 85e4fa8 Roll Dart SDK from e06db8e1b620 to 1d418b40d8bd (1 revision) (flutter/engine#36902)

* 0b57b1c Roll buildroot to a680bb1. (flutter/engine#36901)

* f1634aa Roll Fuchsia Mac SDK from IdQEnRNQNY7ZrLZ04... to jB4jUAxe89I2A-yqv... (flutter/engine#36904)

* f24ea1a Roll Fuchsia Linux SDK from g6-kU8so3PDiR1106... to mdl-0MUwR6uuQdKIm... (flutter/engine#36905)

* 04fa86e Added integration test for platform channels on windows. (flutter/engine#36853)

* 1dbf3ff Convert the executable directory path to UTF-8 on Windows (flutter/engine#36908)

* c3d4fc9 Roll Dart SDK from 1d418b40d8bd to f1d4c7c808bd (2 revisions) (flutter/engine#36913)

* 51b66c9 [Web] Synthesize key events for shift key on pointer events. (flutter/engine#36724)

* 584fffb Roll Fuchsia Mac SDK from jB4jUAxe89I2A-yqv... to fcFu9Z2KJH6oQvHnG... (flutter/engine#36919)

* 4369421 Roll Fuchsia Linux SDK from mdl-0MUwR6uuQdKIm... to NqPnoRHl3WYqH3SrC... (flutter/engine#36920)

* d6d38ab [Impeller] fix null geometry (flutter/engine#36922)

* d7f987e [Impeller] Eliminate unused shader output (flutter/engine#36923)

* c255470 Roll Dart SDK from f1d4c7c808bd to eafe0119c9f5 (2 revisions) (flutter/engine#36925)

* 224a3de Restore support for building the web SDK without a prebuilt Dart SDK (flutter/engine#36926)

* c7c21e5 Re-landing Robolectric 4.8.1 (flutter/engine#34272)

* ab98023 Roll libtess2 to 725e5e08ec8751477565f1d603fd7eb9058c277c (flutter/engine#36928)

* 95e937a Revert "Roll libtess2 to 725e5e08ec8751477565f1d603fd7eb9058c277c (#36928)" (flutter/engine#36932)

* 83092c0 Revert Dart SDK to 2.19.0-324.0.dev (flutter/engine#36930)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App e: impeller

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants