SQ-60/Create location onboarding #211
SQ-60/Create location onboarding #211
Conversation
So we can reuse it in the onboarding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love it, just one small comment to address and it's good for me
android:layout_marginTop="@dimen/onboarding_logo_margin_top" | ||
android:layout_marginEnd="@dimen/onboarding_content_margin_horizontal" | ||
android:contentDescription="@null" | ||
android:src="@drawable/google_primary" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since we already had issues with vector drawables can we just use app:srcCompat
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooops totally forgot :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
@@ -26,51 +27,48 @@ | |||
|
|||
public class Navigator { | |||
|
|||
private final Context context; | |||
private static final int NO_FLAGS = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't there be an empty line between static final
fields and other fields?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...yes there should. Will fix in next PR, takes too much to do a CI build for this XD
This PR creates the basic framework for onboarding and the MVP for the location onboarding.
TODO:
HomeActivity
) to check and re-request permissions/BT as neededProximityFeature
that enables the proximity stuff (including the onboarding!) IIF: