Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 03d7ba5b authored by Massimo Carli's avatar Massimo Carli
Browse files

CompatUI UX Improvements

This CL does the following UX improvements:
  - Increase fading animation duration to 400ms
  - Implement a logic for horizontal and
    vertical education indipendently which displays
    icons when in center position only once and
    every time user double-taps for a not-central
    position.
  - Remove the circles animation

Bug: 275054578
Test: atest WMShellUnitTests:ReachabilityEduLayoutTest
      atest WMShellUnitTests:LetterboxEduWindowManagerTest
      atest WMShellUnitTests:ReachabilityEduWindowManagerTest

Change-Id: I55cd13c83f43a8a31725e6f6493504bc3ab4e62e
parent 30a093f9
Loading
Loading
Loading
Loading
+12 −683

File changed.

Preview size limit exceeded, changes collapsed.

+10 −683

File changed.

Preview size limit exceeded, changes collapsed.

+4 −8
Original line number Original line Diff line number Diff line
@@ -22,48 +22,44 @@
    android:layout_width="match_parent"
    android:layout_width="match_parent"
    android:layout_height="match_parent">
    android:layout_height="match_parent">


    <com.android.wm.shell.compatui.ReachabilityEduHandLayout
    <androidx.appcompat.widget.AppCompatTextView
        style="@style/ReachabilityEduHandLayout"
        style="@style/ReachabilityEduHandLayout"
        android:text="@string/letterbox_reachability_reposition_text"
        android:text="@string/letterbox_reachability_reposition_text"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        android:layout_gravity="center_horizontal|top"
        android:layout_gravity="center_horizontal|top"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:id="@+id/reachability_move_up_button"
        android:id="@+id/reachability_move_up_button"
        android:maxWidth="@dimen/letterbox_reachability_education_item_width"
        android:layout_width="wrap_content"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"/>
        android:layout_height="wrap_content"/>


    <com.android.wm.shell.compatui.ReachabilityEduHandLayout
    <androidx.appcompat.widget.AppCompatTextView
        style="@style/ReachabilityEduHandLayout"
        style="@style/ReachabilityEduHandLayout"
        android:text="@string/letterbox_reachability_reposition_text"
        android:text="@string/letterbox_reachability_reposition_text"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        android:layout_gravity="center_vertical|right"
        android:layout_gravity="center_vertical|right"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:id="@+id/reachability_move_right_button"
        android:id="@+id/reachability_move_right_button"
        android:maxWidth="@dimen/letterbox_reachability_education_item_width"
        android:layout_width="wrap_content"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"/>
        android:layout_height="wrap_content"/>




    <com.android.wm.shell.compatui.ReachabilityEduHandLayout
    <androidx.appcompat.widget.AppCompatTextView
        style="@style/ReachabilityEduHandLayout"
        style="@style/ReachabilityEduHandLayout"
        android:text="@string/letterbox_reachability_reposition_text"
        android:text="@string/letterbox_reachability_reposition_text"
        app:drawableTopCompat="@drawable/reachability_education_ic_left_hand"
        app:drawableTopCompat="@drawable/reachability_education_ic_left_hand"
        android:layout_gravity="center_vertical|left"
        android:layout_gravity="center_vertical|left"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:id="@+id/reachability_move_left_button"
        android:id="@+id/reachability_move_left_button"
        android:maxWidth="@dimen/letterbox_reachability_education_item_width"
        android:layout_width="wrap_content"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"/>
        android:layout_height="wrap_content"/>


    <com.android.wm.shell.compatui.ReachabilityEduHandLayout
    <androidx.appcompat.widget.AppCompatTextView
        style="@style/ReachabilityEduHandLayout"
        style="@style/ReachabilityEduHandLayout"
        android:text="@string/letterbox_reachability_reposition_text"
        android:text="@string/letterbox_reachability_reposition_text"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        app:drawableTopCompat="@drawable/reachability_education_ic_right_hand"
        android:layout_gravity="center_horizontal|bottom"
        android:layout_gravity="center_horizontal|bottom"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:layout_marginTop="@dimen/letterbox_reachability_education_dialog_margin"
        android:id="@+id/reachability_move_down_button"
        android:id="@+id/reachability_move_down_button"
        android:maxWidth="@dimen/letterbox_reachability_education_item_width"
        android:layout_width="wrap_content"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"/>
        android:layout_height="wrap_content"/>


+0 −3
Original line number Original line Diff line number Diff line
@@ -45,9 +45,6 @@
    <!-- Letterbox Dialog -->
    <!-- Letterbox Dialog -->
    <color name="letterbox_dialog_background">@android:color/system_neutral1_900</color>
    <color name="letterbox_dialog_background">@android:color/system_neutral1_900</color>


    <!-- Reachability Education color for hand icon and text-->
    <color name="letterbox_reachability_education_item_color">#BFC8CC</color>

    <!-- GM2 colors -->
    <!-- GM2 colors -->
    <color name="GM2_grey_200">#E8EAED</color>
    <color name="GM2_grey_200">#E8EAED</color>
    <color name="GM2_grey_700">#5F6368</color>
    <color name="GM2_grey_700">#5F6368</color>
+0 −3
Original line number Original line Diff line number Diff line
@@ -330,9 +330,6 @@
    <!-- The margin between the reachability dialog container and its parent. -->
    <!-- The margin between the reachability dialog container and its parent. -->
    <dimen name="letterbox_reachability_education_dialog_margin">16dp</dimen>
    <dimen name="letterbox_reachability_education_dialog_margin">16dp</dimen>


    <!-- The width of each item in the reachability education -->
    <dimen name="letterbox_reachability_education_item_width">118dp</dimen>

    <!-- The size of the icon in the item of reachability education -->
    <!-- The size of the icon in the item of reachability education -->
    <dimen name="letterbox_reachability_education_item_image_size">24dp</dimen>
    <dimen name="letterbox_reachability_education_item_image_size">24dp</dimen>


Loading