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

Commit a628f56d authored by Wenbo Jie's avatar Wenbo Jie
Browse files

[DocsUI M3] Add bottom padding for file area 

In compact screen size, the file area (list/grid) occupies the
full screen height, since in M3 we move breadcrumb down to the
bottom, we need to make extra bottom padding for the file area
to make sure the breadcrumb doesn't over-shadow the last item of
the file area.

Check the attached bug for demo video.

Bug: 397511300
Test: m DocumentsUIGoogle && manual inspection
Flag: com.android.documentsui.flags.use_material3
Change-Id: I2992ef65ec8db7deb77957d777c05bedc0ed4a32
parent a4a406e9

res/flag(com.android.documentsui.flags.use_material3)/layout/drawer_layout.xml

+1 −0
Original line number Diff line number Diff line
@@ -40,6 +40,7 @@ 
            <FrameLayout

                android:layout_width="match_parent"

                android:layout_height="match_parent"

                android:paddingBottom="@dimen/file_area_padding_bottom"

                app:layout_behavior="@string/scrolling_behavior">



                <FrameLayout

res/flag(com.android.documentsui.flags.use_material3)/values/dimens.xml

+4 −0
Original line number Diff line number Diff line
@@ -23,6 +23,10 @@ 
    <dimen name="profile_tab_padding">0dp</dimen>

    <dimen name="grid_container_padding">20dp</dimen>

    <dimen name="list_container_padding">@dimen/space_extra_small_4</dimen>

    <!-- For compact screen, file area occupies the whole screen height, in M3 we show breadcrumb

         at the bottom, so we need to add padding (breadcrumb height) at the bottom to make sure

         breadcrumb won't over-shadow the file area. -->

    <dimen name="file_area_padding_bottom">48dp</dimen>

    <dimen name="icon_size">40dp</dimen>

    <dimen name="button_touch_size">48dp</dimen>

    <dimen name="root_icon_size">24dp</dimen>