Skip to content

Add a breaking change/migration for Paint.enableDithering.#9362

Merged
matanlurey merged 22 commits intoflutter:mainfrom
matanlurey:deprecate-paint-enableDithering
Sep 11, 2023
Merged

Add a breaking change/migration for Paint.enableDithering.#9362
matanlurey merged 22 commits intoflutter:mainfrom
matanlurey:deprecate-paint-enableDithering

Conversation

@matanlurey
Copy link
Contributor

As written. Thanks!

@matanlurey
Copy link
Contributor Author

Apologies, my markdown formatter thrashed the index. I'll fix that right now.

migration guide, but you can find some resources and examples:

- [Writing and using fragment shaders][]
- [`linear_gradient_ssbo_fill.frag`]
Copy link
Contributor

Choose a reason for hiding this comment

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

This is sort of a bad example because it uses features that aren't exposed in the regular fragment program API. A better example might be something like:

https://github.com/jonahwilliams/awesome_gradients/blob/main/shaders/hsl_linear_gradient.frag

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh ok, great, thanks for pointing that out! Fixed.

Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

Does this flag have any impact on the CanvasKit or HTML backends?

@matanlurey
Copy link
Contributor Author

Does this flag have any impact on the CanvasKit or HTML backends?

Good question: I don't believe web ever plumbed dither through.

Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM, but perhaps add a note with a link to any web bugs or caveat that web renderers will behave differently

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

A few edits to tighten the prose and more closely follow the Google Developer Doc Style guidelines.

@matanlurey matanlurey requested a review from sfshaza2 September 7, 2023 21:08
@matanlurey
Copy link
Contributor Author

Friendly ping @sfshaza2.

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm

@matanlurey matanlurey merged commit baef628 into flutter:main Sep 11, 2023
@matanlurey matanlurey deleted the deprecate-paint-enableDithering branch September 11, 2023 18:36
atsansone pushed a commit to atsansone/website that referenced this pull request Sep 19, 2023
…#9362)

As written. Thanks!

---------

Co-authored-by: Shams Zakhour (ignore Sfshaza) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants