16a60dca84
This version is hybrid of the old design and what Andy's suggested. - MessageViewFragment is responsible to show a single message at a time. - Call MVF.openMessage() to tell it to open another message. No need to re-create a new MVF to do this. - MessageView manages the message list, and handles "move-to-newer/older" buttons. Reason for not re-creating a fragment when opening another message is: - Re-using the same fragment doesn't make it as complecated/ugly as I was initially afraid. It's basically cancelling all running tasks, re-initializee some views, and load a message. - We don't have to run MVF.onCreate() over and over again when moving through messages. We may change the strategy later, but I think it's suffice for now. (Changing this might affect how the back key works, so let's revisit it when the fragment manager supports back.) Basically this CL is all about internal changes. No UI should have changed except for: - Moved "Move to newer/older" buttons to the bottom. Also fixes: Bug: 2849129 Change-Id: I00c05069231afded9d98d3d52dd9a7925ebdee9d
72 lines
2.7 KiB
XML
72 lines
2.7 KiB
XML
<?xml version="1.0" encoding="utf-8"?>
|
|
<!-- Copyright (C) 2008 The Android Open Source Project
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
-->
|
|
|
|
<LinearLayout
|
|
xmlns:android="http://schemas.android.com/apk/res/android"
|
|
android:orientation="vertical"
|
|
android:layout_width="match_parent"
|
|
android:layout_height="match_parent"
|
|
android:background="@android:color/white">
|
|
<fragment
|
|
android:id="@+id/message_view_fragment"
|
|
android:name="com.android.email.activity.MessageViewFragment"
|
|
android:layout_width="match_parent"
|
|
android:layout_height="0dip"
|
|
android:layout_weight="1"
|
|
/>
|
|
<!-- Bottom buttons -->
|
|
<LinearLayout
|
|
android:orientation="horizontal"
|
|
android:layout_width="match_parent"
|
|
android:layout_height="wrap_content"
|
|
android:paddingTop="5dip"
|
|
android:paddingLeft="4dip"
|
|
android:paddingRight="4dip"
|
|
android:paddingBottom="1dip"
|
|
android:background="@android:drawable/bottom_bar">
|
|
<Button
|
|
android:id="@+id/reply"
|
|
android:text="@string/reply_action"
|
|
android:layout_height="match_parent"
|
|
android:layout_width="0dip"
|
|
android:layout_weight="1" />
|
|
<Button
|
|
android:id="@+id/reply_all"
|
|
android:text="@string/reply_all_action"
|
|
android:layout_height="match_parent"
|
|
android:layout_width="0dip"
|
|
android:layout_weight="1" />
|
|
<Button
|
|
android:id="@+id/delete"
|
|
android:text="@string/delete_action"
|
|
android:layout_height="match_parent"
|
|
android:layout_width="0dip"
|
|
android:layout_weight="1" />
|
|
<Button
|
|
android:id="@+id/moveToNewer"
|
|
android:text="@string/message_view_move_to_newer"
|
|
android:layout_height="match_parent"
|
|
android:layout_width="0dip"
|
|
android:layout_weight="1" />
|
|
<Button
|
|
android:id="@+id/moveToOlder"
|
|
android:text="@string/message_view_move_to_older"
|
|
android:layout_height="match_parent"
|
|
android:layout_width="0dip"
|
|
android:layout_weight="1" />
|
|
</LinearLayout>
|
|
</LinearLayout>
|