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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 Flyout menu - strange icon (feedback v4.0.0) #487

Closed
WinXaito opened this issue Aug 25, 2022 · 5 comments
Closed

馃悰 Flyout menu - strange icon (feedback v4.0.0) #487

WinXaito opened this issue Aug 25, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@WinXaito
Copy link
Collaborator

Describe the bug

On demo app:

image

There is a icon on the left and the indicator.

And Btw, when we open a submenu, if we click outside the menu, only the submenu is closed (on Windows, when we click outside the menu, all menu is closed). Maybe related to #332)

@bdlukaa
Copy link
Owner

bdlukaa commented Aug 26, 2022

Weird. Is there any chance of it being a rendering issue on the skia side? I can't reproduce it

@WinXaito
Copy link
Collaborator Author

Tested on Firefox + Edge, same problem. (current demo app here https://bdlukaa.github.io/fluent_ui/)

@bdlukaa bdlukaa added the bug Something isn't working label Aug 30, 2022
@bdlukaa
Copy link
Owner

bdlukaa commented Aug 30, 2022

Reproducible on VM-based environments as well!

@jesperborri
Copy link

FlyoutListTile builds the trailing widget twice. Removing line 212 solved it for me.

if (trailing != null)
DefaultTextStyle(
style: TextStyle(
inherit: false,
fontSize: 12.0,
color: theme.borderInputColor,
height: 0.7,
),
child: trailing!,
),
]),
),
if (selected)
PositionedDirectional(
top: 0,
bottom: 0,
child: Container(
margin: const EdgeInsets.symmetric(vertical: 6.0),
width: 2.5,
decoration: BoxDecoration(
color: theme.accentColor.resolveFromReverseBrightness(
theme.brightness,
),
borderRadius: BorderRadius.circular(100),
),
child: trailing!,
),
),
]);

@bdlukaa
Copy link
Owner

bdlukaa commented Dec 11, 2022

Nice catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants