Missing icons in Web built on CI with version 11.0.0

[gdurandrexel]

Describe the bug
When building the release Web version of my App on the CI (clean agent) with version 11.0.0 of the package, all the font-awesome icons are missing.
When I build on my machine, either in debug or release, after a flutter clean, the icons are there.

On the deployed web app, FontManifest.json is found:

and the font are successfully downloaded:

I notice the size of the font files with the v10.12.0 are noticably different:

Also, there is a warning in the console when I open the page that displays the icons, which is not present with v10.12.0:

Is there something I can do to debug this behaviour?

Desktop:

• OS: Windows
• Browser: Edge and Chrome
• Version: latest

Flutter 3.41.5

[gdurandrexel]

I fixed this by building with --no-tree-shake-icons, at the cost of a largely increased size of material icons.

The icons are used in a pretty standard way though, and the other platforms (android, windows) are not affected, e.g.:

IconButton.outlined(
  onPressed: () =>
      showFullscreenDialog<void>(ref, effectiveConfigurationPage),
  icon: const FaIcon(
    FontAwesomeIcons.wrench,
    size: 18,
  ),
  color: iconColor,
  style: buttonStyle,
)

[ghecreagheorghe]

same issue! missing Icons for web on 11.0.0

[narumi147]

@michaelspiss can you please look into this issue? the flutter icon tree shaking didn't work for this package and all icons are gone.

[michaelspiss]

Thanks for testing with the other platforms @gdurandrexel. @dcharkes may I ask whether you are aware of any regressions with the web tree shaking? This seems to be tied to #297 - which worked flawlessly during testing, but I am able to reproduce this problem on a freshly cloned repo. Thanks!

[dcharkes]

Thanks for letting me know @michaelspiss 🙏

Dart2js relies on different logic than native, so that explains why the behavior is different. The icon-tree-shaker is fragile in general, so it doesn't surprise me.

I think the best way to fix this would be to fix-forward (dart-lang/native#2678), but that will take a bit because I need to move record_use to have a stable Dart API first before I use it in Flutter. (Flutter pulls it deps from pub while Dart SDK pulls them in by git hash, and if they misalign we've got troubles.) I'm working on record_use, so I hope to get this done "soon".

[michaelspiss]

Interesting to see how different the build systems are for the web vs. other targets!

Thanks @dcharkes - I'll keep this issue open so that others can see the process