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

Toolbar Icons not Showing on Android via Appshell #5454

Closed
david-maw opened this issue Mar 20, 2022 · 2 comments
Closed

Toolbar Icons not Showing on Android via Appshell #5454

david-maw opened this issue Mar 20, 2022 · 2 comments
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout platform/android 🤖 s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Milestone

Comments

@david-maw
Copy link

Description

On Android if a page has Toolbar items defined and you navigate to it directly from a Shell page the icons do not show but if you navigate to it using Navigation.PushAsync they do show.

Just as an aside the fact that the default toolbar background was white and so were the icons (even though they a specified as black) confused the daylights out of me.

Steps to Reproduce

  1. Unzip the project in MauiShell.zip
  2. Compile and run the project on Android
  3. Navigate to the child page directly from the AppShell page using the hamburger menu, observe the child page shows no toolbar icons
  4. Use the hamburger menu to navigate to the "Main" Page and click to open a child page instance, this time it will show icons (an upper case X, Y and Z).

The pages look like this:

MauiShell

Version with bug

Preview 14 (current)

Last version that worked well

Unknown/Other

Affected platforms

Android, I was not able test on other platforms

Affected platform versions

Android 11

Did you find any workaround?

No

Relevant log output

No response

@david-maw david-maw added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Mar 20, 2022
@Amir-Care Amir-Care added the s/triaged Issue has been reviewed label Mar 21, 2022
@Amir-Care
Copy link

Verified repro with Version 17.2.0 Preview 2.0 [32208.484.main]

@Amir-Care Amir-Care added s/verified Verified / Reproducible Issue ready for Engineering Triage and removed s/triaged Issue has been reviewed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Mar 21, 2022
@jfversluis jfversluis added the area-controls-shell Shell Navigation, Routes, Tabs, Flyout label Mar 21, 2022
@jsuarezruiz jsuarezruiz added this to Ready For Work in Triage Mar 21, 2022
@Redth Redth added this to the 6.0.300-rc.2 milestone Mar 21, 2022
@Redth
Copy link
Member

Redth commented Mar 30, 2022

Tested with our latest main and it appears to work correctly now. This change should be present in the RC1 release.

2022-03-30_10-11-04

@Redth Redth closed this as completed Mar 30, 2022
Triage automation moved this from Ready For Work to Done Mar 30, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout platform/android 🤖 s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

5 participants