[WEB] unusable on 1.26.0-17.0.pre or higher using --web-renderer=html

[jlubeck]

When trying to use flutter_html on flutter 1.26.0-17.0.pre and compiling for web, the page just stays blank and CPU usage goes up until the tab eventually crashes.

Going down to flutter 1.26.0-12.0.pre works fine. So something must have changed between those 2 versions that broken something...

Not sure what else to give for information, as nothing shows up in the logs. Probably an endless loop somewhere?

Let me know if I can help with something else...

[tneotia]

Technically we don't support Flutter Web, so I don't know if we can help out here. It could be some changes with nullsafety in the new Flutter versions? Just throwing darts in the dark here...

[jlubeck]

Yes, I noticed there is no WEB tag on https://pub.dev/packages/flutter_html but it would be a shame to disregard this since it's been working great until now

[tneotia]

I agree. I literally just ported my own package to Flutter Web (much, much harder than I expected), so I would not be opposed to officially porting this one as well now that I have my hands dirty.

One thing that will need to be changed is iframe rendering, as webviews are not supported in Web. This also means conditional importing will be required to prevent webview deps compiling in web, and dart:html compiling in mobile. I see there are a few issues open as well regarding crashes and null exceptions on web. Let me know if there's anything else you know of that would potentially need fixing/changing so I can get a list together and work on this later.

[jlubeck]

Until I run into this issue, I've been using this package on web without issues for almost a year. So I would say I haven't seen anything else that would need huge attention...
Granted, I only use some basic html tags (p, ul, ol, li)

Ohh, wait, I remember. In order to use this I have to force the html renderer. I mentioned this here: #91 (comment)

[ngaurav]

@tneotia Web support is a much needed feature. We hardly use iframes with flutter_html.

[tneotia]

@tneotia Web support is a much needed feature. We hardly use iframes with flutter_html.

Sure, I definitely think this feature would be awesome and lots of people have asked for it. However I don't want to rush it, and having different feature sets for different platforms can be confusing. That's the beauty of Flutter - you get the same functionality across every platform - and we want to uphold that in this package.

I've got a few PRs to update and a bug to fix, which I'm attacking today afternoon. After that if I have some time, I will begin working on something for Flutter Web with nullsafety support. If you've used this package with Flutter Web and encountered any issues, please drop them here so I can look out for them when I am adding support.

[ngaurav]

@tneotia have a good code sprint :) Will definitely do my part with discovering the issues.

[tneotia]

@jlubeck I hastily compiled a web version of the package and it works like a charm, no crashes or high CPU usage.

Flutter doctor

[√] Flutter (Channel beta, 1.26.0-17.6.pre, on Microsoft Windows [Version 10.0.19042.844], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.52.1)
[√] Connected device (2 available)

Can you send over the HTML it crashes with? Or maybe try 17.6 pre?

[jlubeck]

@tneotia Did you try on 1.27.0-4.0.pre ?
Also, I've been seeing your comments on the other tickets that you cannot reproduce the issues... I see you are using Windows and Edge.
I'm using Mac and Chrome.

Might be harder for you to get a hold of a Mac, but can you try on Chrome first?

[tneotia]

I have a mac, let me transfer over there and test mac/chrome. Edge is using Chromium so I don't anticipate much difference but will try regardless.

I did not try 1.27.0-4.0, I'll also try that.

[tneotia]

@jlubeck working well on Channel beta, 1.25.0-8.3.pre with just <p>Test</p>, Chrome macOS

Latest beta is 1.26.0-17.7.pre, working well there as well. (Channel beta, 1.26.0-17.7.pre, on macOS 11.1 20C5061b darwin-x64, locale en-US)

The full-fat example app HTML works fine as well.

[jlubeck]

Yes, channel beta 1.25 is expected to be fine. 1.26.0-17.6.pre was crashing for me though. That was the original title in the bug, which I then changed to the lowest that was still crashing for me. 1.27 is channel dev

[jlubeck]

Even an empty string on data is crashing it for me. I can make a screen recording if you want... but I don't see how that will help you...

[jlubeck]

What flutter_html version are you using @tneotia ? in your pubspec.yml