Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

icons not show on ios (web) #41218

Closed
mahmoudsalah37 opened this issue Sep 24, 2019 · 24 comments
Closed

icons not show on ios (web) #41218

mahmoudsalah37 opened this issue Sep 24, 2019 · 24 comments
Assignees
Labels
customer: webeap platform-ios iOS applications specifically platform-web Web applications specifically waiting for PR to land (fixed) A fix is in flight
Milestone

Comments

@mahmoudsalah37
Copy link

70658878_969441630071656_1026617790720638976_n
work with android and chrome(win 10)

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel unknown, v1.10.2-pre.40, on Microsoft Windows [Version 10.0.17134.1006], locale en-US)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    X Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[√] Chrome - develop for the web
[√] Android Studio (version 3.3)
[√] Android Studio (version 3.5)
[√] VS Code (version 1.38.1)
[√] Connected device (2 available)

! Doctor found issues in 1 category.
PS E:\QUALITY 2\iso_flutter> flutter doctor -v
[√] Flutter (Channel unknown, v1.10.2-pre.40, on Microsoft Windows [Version 10.0.17134.1006], locale en-US)
    • Flutter version 1.10.2-pre.40 at C:\flutter\flutter
    • Framework revision b4ab5cfe79 (8 days ago), 2019-09-16 20:34:25 -0700
    • Engine revision 7ea9884ab0
    • Dart version 2.5.0 (build 2.5.0-dev.4.0 be66176534)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\flutter\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\flutter\sdk
    • Java binary at: C:\flutter\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b01)
    X Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] Android Studio (version 3.5)
    • Android Studio at C:\flutter\android-studio
    • Flutter plugin version 34.0.4
    • Dart plugin version 191.6183.88
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b01)

[√] VS Code (version 1.38.1)
    • VS Code at C:\Users\Mahmoud salah eldin\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] Connected device (2 available)
    • Chrome • chrome • web-javascript • Google Chrome 76.0.3809.132
    • Server • web    • web-javascript • Flutter Tools

! Doctor found issues in 1 category.
@timtraversy
Copy link
Contributor

timtraversy commented Sep 24, 2019

Hey! I believe this is a duplicate of #40501. The fix was added to the engine a week ago in version 1.10.5, you just need to upgrade Flutter to at least that version.

Actually scratch that, because I'm seeing the same thing now.

@escamoteur
Copy link
Contributor

Is this picture flutter Web on a mobile browser?

@timtraversy
Copy link
Contributor

Pretty sure it's mobile Safari.

@twpochuan
Copy link

Same issue here.
I've tested different flutter web demos on different iOS 13 devices. All of them having the same problem despite their browers. However, it was fine on iOS 12.

@jonahwilliams
Copy link
Member

cc @hterkelsen. Is it possible that our browser detection isn't working on newer iOS devices?

@jonahwilliams jonahwilliams added the platform-web Web applications specifically label Sep 25, 2019
@escamoteur escamoteur added the platform-ios iOS applications specifically label Sep 25, 2019
@mahmoudsalah37
Copy link
Author

Is this picture flutter Web on a mobile browser?

yes real device

@mahmoudsalah37
Copy link
Author

Same issue here.
I've tested different flutter web demos on different iOS 13 devices. All of them having the same problem despite their browers. However, it was fine on iOS 12.

yes this phone ios 13

@nturgut
Copy link
Contributor

nturgut commented Sep 26, 2019

Can you run flutter on profile or release mode? Such as:

flutter run -d web --profile

@nturgut nturgut added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label Sep 26, 2019
@mahmoudsalah37
Copy link
Author

mahmoudsalah37 commented Sep 27, 2019

flutter run -d web --profile

(yes i can)
flutter run -d web --profile
Launching lib\main.dart on Server in profile mode...
lib\main.dart is being served at http://localhost:51188/
Building application for the web... 18.1s

👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻
Warning: Flutter's support for building web applications is highly experimental.
For more information see #34082.
👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻 👻

🔥 To hot restart (and rebuild state), press "R".
For a more detailed help message, press "h". To quit, press "q".

@no-response no-response bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label Sep 27, 2019
@rodydavis
Copy link
Contributor

are there any updates or temporary workarounds?

@nturgut
Copy link
Contributor

nturgut commented Oct 2, 2019

This is a safari issue. I managed to produce it both: Safari on IOS13 device and Safari on Mac.

Screen Shot 2019-10-02 at 9 35 53 AM

@nturgut nturgut added this to the Near-term Goals milestone Oct 2, 2019
@nturgut
Copy link
Contributor

nturgut commented Oct 2, 2019

/cc @hterkelsen

@rodydavis
Copy link
Contributor

Does the new update to safari require something other than a font manifest now?

@rxlabz
Copy link
Contributor

rxlabz commented Oct 2, 2019

Curiously, switching to Safari "responsive mode" (Ctrl+Cmd+R) seems to enable the Roboto font, but not the Material icons...🤷‍♂️

@rodydavis
Copy link
Contributor

Very weird 🧐

@nturgut nturgut self-assigned this Oct 3, 2019
@nturgut nturgut modified the milestones: Near-term Goals, October 2019 Oct 3, 2019
@nturgut
Copy link
Contributor

nturgut commented Oct 3, 2019

/cc @hterkelsen The solution you suggested works.

As a temporary work around, one can remove webkit from this check:

https://github.com/flutter/engine/blob/master/lib/web_ui/lib/src/engine/text/font_collection.dart#L121

Meanwhile I will investigate in which Safari versions might also cause problem.

@rodydavis
Copy link
Contributor

OK perfect! Thank you

@rlechetaudemy
Copy link

rlechetaudemy commented Oct 6, 2019

font_awesome_flutter plugin works well in Chrome too, but not in Safari.

@nturgut nturgut added the waiting for PR to land (fixed) A fix is in flight label Oct 10, 2019
@nturgut
Copy link
Contributor

nturgut commented Oct 11, 2019

The PR is in.

@nturgut nturgut closed this as completed Oct 11, 2019
@rodydavis
Copy link
Contributor

Thank you!!

@hiroshihorie
Copy link

I don't know why it doesn't seem to be fixed... 🥺
I'm using master channel.

@nturgut
Copy link
Contributor

nturgut commented Oct 12, 2019

Hi,

Last engine roll is 8 hours ago, so the fix PR is only in flutter/engine repository now. It will probably roll it to flutter in a few hours.

One alternative is you can sync the engine and run local engine:

flutter run -d headless-server --local-engine=host_debug_unopt --web-port=8080

@rodydavis
Copy link
Contributor

Really appreciate the fast fix! Thanks again guys 🚀

@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
customer: webeap platform-ios iOS applications specifically platform-web Web applications specifically waiting for PR to land (fixed) A fix is in flight
Projects
None yet
Development

No branches or pull requests

10 participants