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

FontVariation.lerp, custom FontVariation constructors, and more documentation #43750

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

Hixie
Copy link
Contributor

@Hixie Hixie commented Jul 17, 2023

This should aid with implementing the framework side of flutter/flutter#105120 This should also address flutter/flutter#28543.

@Hixie Hixie force-pushed the fontweight branch 11 times, most recently from 3c83dad to 586a63c Compare July 20, 2023 07:13
@github-actions github-actions bot added the platform-web Code specifically for the web engine label Jul 20, 2023
…entation

This should aid with implementing the framework side of flutter/flutter#105120
This should also address flutter/flutter#28543.
Copy link
Contributor

@eyebrowsoffire eyebrowsoffire left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for adding all this documentation!

@feinstein
Copy link

@eyebrowsoffire when you approved it, did you see my previous post? I think there's a typo.

@Hixie
Copy link
Contributor Author

Hixie commented Aug 23, 2023

@feinstein Can you point out the typo? I'm not sure where you mean.

@feinstein
Copy link

@Hixie I did on my previous comment, it says meigh, I think it should be weigh, unless that's a term I am not familiar with.

auto-submit bot pushed a commit that referenced this pull request Aug 23, 2023
@Hixie
Copy link
Contributor Author

Hixie commented Aug 23, 2023

@feinstein Thanks. I don't see your earlier comment anywhere, but I shall endeavour to include a fix when I reland this patch (it had to be reverted for other reasons)!

@Hixie
Copy link
Contributor Author

Hixie commented Aug 23, 2023

Follow-up in #44996.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 23, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 23, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 23, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 23, 2023
…133139)

flutter/engine@7d56840...72a0642

2023-08-23 skia-flutter-autoroll@skia.org Roll Skia from ee7404db8273 to d0c369eaa637 (1 revision) (flutter/engine#44997)
2023-08-23 skia-flutter-autoroll@skia.org Roll Skia from d0d2b7042bb9 to ee7404db8273 (1 revision) (flutter/engine#44995)
2023-08-23 skia-flutter-autoroll@skia.org Roll Skia from eac8f8ea2660 to d0d2b7042bb9 (1 revision) (flutter/engine#44994)
2023-08-23 zanderso@users.noreply.github.com Revert "FontVariation.lerp, custom FontVariation constructors, and more documentation" (flutter/engine#44993)
2023-08-23 skia-flutter-autoroll@skia.org Roll Skia from aa208c8a2d60 to eac8f8ea2660 (1 revision) (flutter/engine#44992)
2023-08-23 dkwingsmt@users.noreply.github.com Move Rasterizer::Draw's discard_callback to Delegate (flutter/engine#44813)
2023-08-23 ian@hixie.ch FontVariation.lerp, custom FontVariation constructors, and more documentation (flutter/engine#43750)
2023-08-23 ian@hixie.ch Make web tests start with an empty title (flutter/engine#43846)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
auto-submit bot pushed a commit that referenced this pull request Aug 23, 2023
…entation (#44996)

This should aid with implementing the framework side of flutter/flutter#105120.
This should also address flutter/flutter#28543.

This is a reland of #43750 with two changes, one to fix a typo mentioned in #43750 (comment), and one to fix the analysis error found when rolling this to the framework (https://ci.chromium.org/ui/p/flutter/builders/try/Linux%20analyze/62845/overview). The latter change is temporary and can be relaxed when FontVariations is reexported from dart:ui. I plan to do that when fixing flutter/flutter#105120.
auto-submit bot pushed a commit that referenced this pull request Aug 25, 2023
…entation (#45030)

This should aid with implementing the framework side of flutter/flutter#105120.
This should also address flutter/flutter#28543.

Original commit in #43750 was reverted in #44993.
Second attempt in #44996 was reverted in #45023.
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
…entation (flutter#43750)

This should aid with implementing the framework side of flutter/flutter#105120 This should also address flutter/flutter#28543.
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
…entation (flutter#44996)

This should aid with implementing the framework side of flutter/flutter#105120.
This should also address flutter/flutter#28543.

This is a reland of flutter#43750 with two changes, one to fix a typo mentioned in flutter#43750 (comment), and one to fix the analysis error found when rolling this to the framework (https://ci.chromium.org/ui/p/flutter/builders/try/Linux%20analyze/62845/overview). The latter change is temporary and can be relaxed when FontVariations is reexported from dart:ui. I plan to do that when fixing flutter/flutter#105120.
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
…entation (flutter#45030)

This should aid with implementing the framework side of flutter/flutter#105120.
This should also address flutter/flutter#28543.

Original commit in flutter#43750 was reverted in flutter#44993.
Second attempt in flutter#44996 was reverted in flutter#45023.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects: engine autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants