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

Conversation

@CoderDake
Copy link

@CoderDake CoderDake commented Oct 10, 2023

Adds Noto Music font as a fallback font, and updates DEPS and font definitions after running the font roller script.

Issues

Fixes: flutter/devtools#6149


I used the developer API to make sure the api works with this font.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the platform-web Code specifically for the web engine label Oct 10, 2023
@ditman ditman changed the title Add Noto Music to the list of fallback fonts [web] Adds Noto Music fallback font Oct 10, 2023
@ditman ditman marked this pull request as ready for review October 10, 2023 22:18
@flutter-dashboard

This comment was marked as resolved.

@ditman
Copy link
Member

ditman commented Oct 10, 2023

This should be good to go, PTAL web team!

Copy link
Contributor

@harryterkelsen harryterkelsen left a comment

Choose a reason for hiding this comment

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

LGTM

@ditman
Copy link
Member

ditman commented Oct 10, 2023

info - test/ui/fallback_fonts_golden_test.dart:172:18 - The local variable '_checkDownloadedFamiliesForString' starts with an underscore. Try renaming the variable to not start with an underscore. - no_leading_underscores_for_local_identifiers

Khaaaaannnn!

@ditman ditman added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 11, 2023
@CoderDake
Copy link
Author

Fixes flutter/devtools#6149

@auto-submit auto-submit bot merged commit 97bcbd5 into flutter:main Oct 11, 2023
@ditman ditman self-assigned this Oct 11, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 11, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 11, 2023
…136414)

flutter/engine@2b1b4b9...8bf1460

2023-10-11 [email protected] Revert "[Android] Add support for text processing actions (#44579)" (flutter/engine#46788)
2023-10-11 [email protected] [web] Adds Noto Music fallback font (flutter/engine#46720)
2023-10-11 [email protected] Fix some image_external_texture entries in the license golden file (flutter/engine#46790)
2023-10-11 [email protected] Rename HardwareBufferExternalTexture* to ImageExternalTexture* (flutter/engine#46786)

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 [email protected],[email protected],[email protected] 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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
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 platform-web Code specifically for the web engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some characters not displayed correctly on web

4 participants