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

Change the flyout icon color #6738

Open
AlexeyZhukoff opened this issue May 2, 2022 · 16 comments · May be fixed by #27580
Open

Change the flyout icon color #6738

AlexeyZhukoff opened this issue May 2, 2022 · 16 comments · May be fixed by #27580
Assignees
Labels
area-controls-flyoutpage FlyoutPage partner/syncfusion Issues / PR's with Syncfusion collaboration platform/iOS 🍎 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@AlexeyZhukoff
Copy link

Description

How can i change flyout icon color? It's always blue, but in orinal file i have black color

Steps to Reproduce

  1. Download example app
  2. Run it
  3. You see blue flyout icon
    Archive.zip

Version with bug

Release Candidate 2 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 15.4

Did you find any workaround?

no

Relevant log output

No response

@AlexeyZhukoff AlexeyZhukoff added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels May 2, 2022
@Redth Redth added the area-controls-flyoutpage FlyoutPage label May 2, 2022
@Redth Redth added this to the 6.0.300-servicing milestone May 2, 2022
@Redth
Copy link
Member

Redth commented May 2, 2022

Try setting your own custom image source for the flyout icon?

@Redth Redth added the s/needs-info Issue needs more info from the author label May 2, 2022
@ghost
Copy link

ghost commented May 2, 2022

Hi @AlexeyZhukoff. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@hartez hartez changed the title Change the flayout icon color Change the flyout icon color May 2, 2022
@jfversluis jfversluis removed the s/needs-verification Indicates that this issue needs initial verification before further triage will happen label May 2, 2022
@AlexeyZhukoff
Copy link
Author

Hello, in example project i set my custom image for flyout icon, but icon always tint to blue color.

@ghost ghost added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels May 3, 2022
@AlexeyZhukoff
Copy link
Author

Try setting your own custom image source for the flyout icon?
Hello, in example project i set my custom image for flyout icon, but icon always tint to blue color.

@XamlTest XamlTest added the s/verified Verified / Reproducible Issue ready for Engineering Triage label May 5, 2022
@XamlTest
Copy link

XamlTest commented May 5, 2022

Verified this issue with Visual Studio Enterprise 17.3.0 Preview 1.0 [32427.455.main]. Repro on iOS 15.4. Sample Project: MauiApp2.zip

@lnaie
Copy link

lnaie commented May 23, 2022

I see this happening as well in the latest maui with the default vs2022 preview template. If I keep it pressed eventually there is an indication that the hamburger icon is there. How do I fix its color/visibility in the default state?

@lnaie
Copy link

lnaie commented May 23, 2022

I see this happening as well in the latest maui with the default vs2022 preview template. If I keep it pressed eventually there is an indication that the hamburger icon is there. How do I fix its color/visibility in the default state?

Ok, It got fixed in the latest, I've just installed. Thanks!

@AntonKosenkoDX
Copy link

@Redth Hello! Any update on this issue?

@Redth Redth modified the milestones: 6.0-servicing, Backlog Aug 30, 2022
@ghost
Copy link

ghost commented Aug 30, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@samhouts samhouts removed the s/verified Verified / Reproducible Issue ready for Engineering Triage label Apr 5, 2023
@Zhanglirong-Winnie Zhanglirong-Winnie added s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage labels Apr 26, 2023
@Zhanglirong-Winnie
Copy link

Zhanglirong-Winnie commented Apr 26, 2023

Verified this issue with Visual Studio Enterprise 17.6.0 Preview 2.0. Repro on iOS simulator with sample project.
Archive.zip
image

@Qythyx
Copy link

Qythyx commented Jun 13, 2023

@Redth , it's approaching 1 year since this was moved to the backlog. Any chance this will be picked up soon?

@markkellaway
Copy link

@Redth , it's approaching 1 year since this was moved to the backlog. Any chance this will be picked up soon?

+1
This issue is causing us headaches, having to use the default hamburger menu is not a good workaround for us.

@HobDev
Copy link

HobDev commented Oct 17, 2023

Facing this issue on Windows. I am developing with .NET 8 RC2

@Thaldoras
Copy link

Thaldoras commented Jun 28, 2024

We use a FlyoutPage that has a TabbedPage as Detail.
Inside each Tab is a NavigationPage.

To change the color of the back button in a NavigationPage you can use
NavigationPage.SetIconColor(this, AppColours.PrimaryDark);

We make that call in the base class for all out pages. By some kind of black magic this will update the color of the burger menu icon (flyout menu icon) on Android.

For iOS it will not update the color of the flyout icon until you have first navigated to a page inside one of the tabs (a page that displays the back button) and then navigated back.

TLDR: if you have a NavigationPage inside your FlyoutPage detail then you can use the above method to change the icon color.

@Thaldoras
Copy link

Following up. You can use the NavagationPage BarTextColor property to set the color of the icon.

@MauiUIui
Copy link

MauiUIui commented Nov 1, 2024

This seems like a simple issue to solve, yet nobody is doing anything about it for two years? Really? This effects the design greatly.

Well, in case it is not clear here is a summery that might help fixing:
When no custom FlyoutIcon is selected, the color can be changed by changing the Shell.ForegroundColor, which by the way also effects the color of the BackButton.
However as soon as you want to have your own HamburgerMenu look, which I assume almost everybody would want, you have no control over the color, as it apparently becomes white (on my android at least, where the android specific colorPrimary, colorPrimaryDark and
colorAccent colors aren't set to white.
My proposals for a solution:
1- Do you thing by correcting this, probably by making a property (preferably of type Brush) to control this color.
2- Change the type of the Shell.FlyoutIcon property from ImageSource to Image so that we could use the community toolkit's IconTintColorBehavior to change the TintColor (not perfect, but would work for most people).
3- Keep ignoring us and let's die using the dull default hamburger menu image, just so we could control the color and not break our UI theme.

EDIT: Oh yea, one other very important thing: If you dare changing the FlyoutIcon you end up without a BackButton, as the chosen FlyoutIcon is frozen there forever.
EDIT2: I should probably change suggestion 3 with a more serious one: Kill the FlyoutIcon property all together and put us all out of our misery.

@vishnumenon2684 vishnumenon2684 added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Feb 4, 2025
@vishnumenon2684 vishnumenon2684 self-assigned this Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-flyoutpage FlyoutPage partner/syncfusion Issues / PR's with Syncfusion collaboration platform/iOS 🍎 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet