Open hellais opened 5 months ago
Feedback:
🔴 == essential feedback (p0) 🟡 == important feedback (p1) 🟦 == nice to have (p2)
The following release addresses all the above points of feedback: https://appdistribution.firebase.google.com/testerapps/1:951667061699:android:13ca15ecce8c0f15374376/releases/022gojfbe7588?utm_source=firebase-console
🔴 When I first opened the app every test card except Websites and Instant Messaging was greyed out not allowing me to tap on them. Now everything except Websites is greyed out.
I tested the above release and there is still something broken here.
I ran a few tests and for some unexplainable reason one of the cards became deselected and toggled off.
Do you know what's up with that?
Edit: I think I understand now what the issue is. For some reason the selection of the tests in the run all tests view affects which cards are tappable and selected or not in the dashboard.
I think we have removed since several revisions of the designs the concept of a card being disabled in the dashboard, so all code related to this feature should be removed and this bug should be fixed.
🟡 I think the read more option can include more text before you need to truncate it. On my phone screen the text inside of the websites card fits well within the screen allowing me to see the buttons under it too.
This still needs some tweaking. I think that for the default OONI tests it's not needed to have the read more.
🟦 Related to this, we may want to have a nicer pattern for the expand/collapse by using a style that doesn't conflict with the style we are already using for links
This is also not addressed, it has stayed the same.
This is the issue to track feedback for: https://github.com/ooni/probe-android/pull/636