markgravity / flutter-survey

3 stars 0 forks source link

[62] [UI] Side Menu screen #67

Closed markgravity closed 3 years ago

markgravity commented 3 years ago

https://github.com/markgravity/flutter-survey/issues/62

What happened

Create UI for Side Menu screen

Proof Of Work

sleepylee commented 3 years ago

the survey image in the background looks quite pixelated 🤔 I thought you already updated it with _l? 🤔

manh-t commented 3 years ago

@markgravity I think the correct behavior of side menu should be:

Besides, SliderMenuContainer is parent of Screen (Scaffold), but I think the side menu should be a part of screen tree instead. How do you think? 🙏

Please correct me if I'm wrong 🙇

https://user-images.githubusercontent.com/60863885/122203039-d73f5d80-cec7-11eb-87e1-ff54d427cfa7.mp4

markgravity commented 3 years ago
  • When we open menu, it should show on top of its screen (it likes a stack, the screen is behind the side menu), and the screen position should not changed. I observe that you are putting the screen to the left when opening side menu which is not good UI/ UX. You may refer to iOS Gmail app to see typical behavior when tapping Hamburger menu to open side menu. 😄

Just skip it just now, please. 🙏

  • The screen should not be interacted while side menu is opening.
  • Tapping outside of side menu to dismiss it, or swipe to the right to dismiss the side menu (optional)

I added it Integrate task, I just try to show the UI here 😁

Besides, SliderMenuContainer is parent of Screen (Scaffold), but I think the side menu should be a part of screen tree instead. How do you think? 🙏

I think that better, let me update it after the Test PR to prevent conflict 😁