blallo
/
Signal-Android
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix overdraw issues in the conversation view 

Remove the redundant window background and row backgrounds to improve
drawing performance.
master
Veeti Paananen 2014-03-04 11:47:27 +02:00
parent dc77c43435
commit 42aeca26f1
7 changed files with 18 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
res/layout/conversation_activity.xml
View File

@ -3,6 +3,7 @@
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="fill_parent"
android:background="?conversation_background"
android:orientation="vertical">
<org.thoughtcrime.securesms.components.RecipientsPanel
@ -17,7 +18,6 @@
android:layout_height="fill_parent"
android:layout_weight="1"
android:orientation="vertical"
android:background="?conversation_background"
android:gravity="bottom">
<fragment
@ -33,8 +33,7 @@
android:id="@id/bottom_container"
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:orientation="vertical"
android:background="?conversation_background">
android:orientation="vertical">
<ScrollView android:layout_width="fill_parent"
android:layout_height="wrap_content"

3
res/layout/conversation_fragment.xml
View File

@ -18,6 +18,7 @@
android:fadingEdge="none"
android:divider="@android:color/transparent"
android:dividerHeight="0dp"
android:layout_marginBottom="1dip"/>
android:layout_marginBottom="1dip"
android:cacheColorHint="?conversation_background" />
</LinearLayout>

1
res/layout/conversation_item_activity.xml
View File

@ -5,7 +5,6 @@
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:orientation="vertical"
android:background="?conversation_background"
android:padding="8dp">
<TextView android:id="@+id/conversation_item_body"

1
res/layout/conversation_item_received.xml
View File

@ -4,7 +4,6 @@
android:layout_height="wrap_content"
android:paddingRight="10dip"
android:orientation="vertical"
android:background="?conversation_background"
xmlns:android="http://schemas.android.com/apk/res/android">
<TextView android:id="@+id/group_message_status"

3
res/layout/conversation_item_sent.xml
View File

@ -4,8 +4,7 @@
android:id="@+id/conversation_item"
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:orientation="horizontal"
android:background="?conversation_background">
android:orientation="horizontal">
<RelativeLayout android:layout_width="match_parent"
android:layout_height="wrap_content"

10
res/values/themes.xml
View File

@ -145,6 +145,16 @@
<item name="navigation_drawer_shadow">@drawable/drawer_shadow_dark</item>
</style>
<style name="TextSecure.LightTheme.ConversationActivity"
parent="@style/TextSecure.LightTheme">
<item name="android:windowBackground">@null</item>
</style>
<style name="TextSecure.DarkTheme.ConversationActivity"
parent="@style/TextSecure.DarkTheme">
<item name="android:windowBackground">@null</item>
</style>
<style name="TextSecure.Light.Dialog"
parent="@android:style/Theme.Dialog"
tools:ignore="NewApi">

3
src/org/thoughtcrime/securesms/util/DynamicTheme.java
View File

@ -6,6 +6,7 @@ import android.os.Build;
import android.preference.PreferenceManager;
import org.thoughtcrime.securesms.ApplicationPreferencesActivity;
import org.thoughtcrime.securesms.ConversationActivity;
import org.thoughtcrime.securesms.ConversationListActivity;
import org.thoughtcrime.securesms.R;
@ -41,9 +42,11 @@ public class DynamicTheme {
if (theme.equals("light")) {
if (activity instanceof ConversationListActivity) return R.style.TextSecure_LightTheme_NavigationDrawer;
else if (activity instanceof ConversationActivity) return R.style.TextSecure_LightTheme_ConversationActivity;
else return R.style.TextSecure_LightTheme;
} else if (theme.equals("dark")) {
if (activity instanceof ConversationListActivity) return R.style.TextSecure_DarkTheme_NavigationDrawer;
else if (activity instanceof ConversationActivity) return R.style.TextSecure_DarkTheme_ConversationActivity;
else return R.style.TextSecure_DarkTheme;
}