-
Notifications
You must be signed in to change notification settings - Fork 1
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
[#27][#28][UI][Integrate] As a user, I can see multiple selects answer #71
Conversation
@hoangnguyen92dn as I check the PoW, we cannot tick the checkbox if its index is not selected. With this implementation, we have to scroll to the item, then tick its checkbox to select the option, which is complex in term of UX 😂 . Can we update it to:
|
@manh-t The design confused me with the unselected item so I thought the behavior is similar to the dropdown answer. Let me update then. |
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## develop #71 +/- ##
===========================================
- Coverage 33.53% 32.55% -0.98%
===========================================
Files 72 76 +4
Lines 1163 1244 +81
===========================================
+ Hits 390 405 +15
- Misses 773 839 +66
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes
|
Close #27
Close #28
What happened 👀
Add multiple choices answer view type
Insight 📝
pick
field from API response to determine single/multiple choicesNote: I tried the ListWheelScrollView but the CheckBox doesn't notify the state changes
Proof Of Work 📹
SVID_20230420_142507_1.mp4
Integration test passed ✅