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

[Gallery][Android]Flyout(Material) For "Help" section when choose the Nested Flyout for "Settings" the gallery app gets freeze. #6934

Closed
sakshi173 opened this issue Sep 1, 2021 · 7 comments
Labels
area/canary 🐤 Categorizes an issue or PR as relevant to canary testing area/popups 💬 Categorizes an issue or PR as relevant to popups or flyouts difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..) project/themes-toolkit-gallery Categorizes an issue or PR as relevant to Uno.Themes\Toolkit\Gallery

Comments

@sakshi173
Copy link

Current behavior

20210901_140245.mp4

Expected behavior

The Nested flyout for Settings should open properly and app should not get freeze.

How to reproduce it (as minimally and precisely as possible)

  1. Open Uno Gallery APP for canary.
  2. Click "Flyout" option from Burger Menu.
  3. Click “Help” Button in “Nested MenuFlyout ” section.
  4. Click/Tap the "Send Feedback" section.
  5. Click/Tap the "Settings" section.
  6. Observe that the app freeze and tap/click is not working.

Workaround

No response

Works on UWP/WinUI

No response

Environment

No response

NuGet package version(s)

No response

Affected platforms

Android

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

Canary Build
1.5.0 (400825)
Aug 31, 2021

@sakshi173 sakshi173 added kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform triage/untriaged Indicates an issue requires triaging or verification area/canary 🐤 Categorizes an issue or PR as relevant to canary testing difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Sep 1, 2021
@jeromelaban jeromelaban added area/popups 💬 Categorizes an issue or PR as relevant to popups or flyouts project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..) labels Sep 2, 2021
@Youssef1313
Copy link
Member

Related to #4795.

@Xiaoy312
Copy link
Contributor

Xiaoy312 commented Jan 19, 2022

This happens because nested menu-flyout is unable to light-dismiss:
Given this setup:

           ┌────────┐
[Button]──►│T1 menu │
           ├────────┤
           │T1 item1│  ┌────────┐
           │T1 item2├─►│T2 menu │
           └────────┘  ├────────┤
                       │T2 item1│
                       │T2 item2│
                       └────────┘

when the T1 (tier 1) menu is open, you can still dismiss by clicking any empty space, or complete by pressing the menu items
however, when the nested T2 menu is open, you can no longer light-dismiss by clicking any empty space, or interact with previous menu, but it is still possible to complete by pressing the t2 menu items

the problem here is that, the T3 menu is opened offscreen, effectively locking the option to complete the flyout
you can use the device's native home button or (droid-only)back button to get out of this situation, as alt-tabbing will close any existing flyout

note: normally the flyout should be opened on the left, if the right side will make it offscreen
edit: this issue seems to only manifest on android, other platforms (uwp, ios, wasm) are working correctly

@jeromelaban
Copy link
Member

We've made some recent changes, @sakshi173 could you revalidate?

@sakshi173 sakshi173 changed the title [Flyout] For "Help" section when choose the Nested Flyout for "Settings" the gallery app gets freeze. [Gallery][Android]Flyout(Material) For "Help" section when choose the Nested Flyout for "Settings" the gallery app gets freeze. Mar 9, 2022
@sakshi173
Copy link
Author

sakshi173 commented Mar 9, 2022

@jeromelaban , I revalidated it, and found it, the issue is happening in the below steps.
1.Launch app and go to 'Flyout' Section.
2. Now first time click on 'Settings' options.
3. Notice that the screen is getting hanged.
4. Now tap mobile's back button(2 times)
5. Tap again the 'Settings' option.
6. Notice that now the flyout menu works fine.
Below is the video for the same steps.

flyout.issue_Trim.mp4

@sakshi173 sakshi173 added the project/themes-toolkit-gallery Categorizes an issue or PR as relevant to Uno.Themes\Toolkit\Gallery label Jul 19, 2022
@sakshi173
Copy link
Author

@jeromelaban, I verified this issue is still present.
Steps to reproduce the bug are below.
1.Launch app and go to 'Flyout' Section.
2. Now first time click on 'Settings' options.
3. Notice that the screen is getting hanged.
4. Now tap mobile's back button(2 times)
5. Tap again the 'Settings' option.
6. Notice that now the flyout menu works fine.

@FrozDark
Copy link

Possible related to the size of flyout
#11597

@MartinZikmund MartinZikmund added difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers and removed triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Jul 14, 2023
@nguyeenvy
Copy link

This issue seems resolved in the latest Uno Gallery app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/canary 🐤 Categorizes an issue or PR as relevant to canary testing area/popups 💬 Categorizes an issue or PR as relevant to popups or flyouts difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform project/layout 🧱 Categorizes an issue or PR as relevant to layouting and containers (Measure/Arrange, Collections,..) project/themes-toolkit-gallery Categorizes an issue or PR as relevant to Uno.Themes\Toolkit\Gallery
Projects
None yet
Development

No branches or pull requests

7 participants