chrome/android/java/res/layout/tabular_context_menu_page.xml - Issue 2777773002: Show the image header for the Context Menu

Side by Side Diff: chrome/android/java/res/layout/tabular_context_menu_page.xml

Issue 2777773002: Show the image header for the Context Menu (Closed)

Patch Set: Created 3 years, 9 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

« chrome/android/java/res/drawable/tabular_context_menu_image_border.xml ('K') | « chrome/android/java/res/drawable/tabular_context_menu_image_border.xml ('k') | chrome/android/java/res/values-sw600dp/dimens.xml » ('j') | chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <?xml version="1.0" encoding="utf-8"?>	1 <?xml version="1.0" encoding="utf-8"?>

2 <!-- Copyright 2017 The Chromium Authors. All rights reserved.	2 <!-- Copyright 2017 The Chromium Authors. All rights reserved.

3 Use of this source code is governed by a BSD-style license that can be	3 Use of this source code is governed by a BSD-style license that can be

4 found in the LICENSE file. -->	4 found in the LICENSE file. -->

5 <LinearLayout	5 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

6 xmlns:android="http://schemas.android.com/apk/res/android"	6 xmlns:tools="http://schemas.android.com/tools"

7 android:id="@+id/content_layer"	7 android:id="@+id/content_layer"

8 android:layout_width="match_parent"	8 android:layout_width="match_parent"

9 android:layout_height="match_parent"	9 android:layout_height="match_parent"

10 android:minWidth="280dp"	10 android:minWidth="280dp"

11 android:orientation="vertical">	11 android:orientation="vertical">

12 <TextView	12 <LinearLayout

13 android:id="@+id/context_header_text"	13 android:id="@+id/context_header_layout"

14 android:layout_width="match_parent"	14 android:layout_width="match_parent"

15 android:layout_height="wrap_content"	15 android:layout_height="wrap_content"

16 android:layout_marginTop="20dp"	16 android:layout_marginTop="20dp"

17 android:layout_marginBottom="10dp"

18 android:layout_marginStart="20dp"	17 android:layout_marginStart="20dp"

19 android:layout_marginEnd="20dp"	18 android:layout_marginEnd="20dp"

20 android:gravity="center"	19 android:layout_marginBottom="10dp"

21 android:maxLines="1"	20 android:layout_gravity="center"

22 android:textSize="13sp"	21 android:orientation="vertical"

23 android:textStyle="bold"	22 tools:ignore="UseCompoundDrawables">
	JJ 2017/03/27 00:38:54 For those who are curious on why we're doing this: For those who are curious on why we're doing this: So basically a compund drawable could work if we could set the background and max height but since we can't we have to put this here. Using a linear layout around it allows us to apply the margins regardless of what is inside the views. Theresa 2017/03/27 18:28:26 It would be good to capture this in a comment in t Show quoted text On 2017/03/27 00:38:54, JJ wrote: > For those who are curious on why we're doing this: So basically a compund > drawable could work if we could set the background and max height but since we > can't we have to put this here. > > Using a linear layout around it allows us to apply the margins regardless of > what is inside the views. It would be good to capture this in a comment in the XML so that we remember why it wasn't implemented as a compound drawable. JJ 2017/03/27 20:47:12 Done. Show quoted text On 2017/03/27 18:28:26, Theresa wrote: > On 2017/03/27 00:38:54, JJ wrote: > > For those who are curious on why we're doing this: So basically a compund > > drawable could work if we could set the background and max height but since we > > can't we have to put this here. > > > > Using a linear layout around it allows us to apply the margins regardless of > > what is inside the views. > > It would be good to capture this in a comment in the XML so that we remember why > it wasn't implemented as a compound drawable. Done.
24 android:ellipsize="end"	23 <ImageView

25 android:visibility="gone"/>	24 android:id="@+id/context_header_image"

	25 android:layout_width="wrap_content"

	26 android:layout_height="wrap_content"

	27 android:layout_gravity="center"

	28 android:contentDescription="@string/context_menu_header_image_defaul t_desc_text"

	29 android:foreground="@drawable/tabular_context_menu_image_border"

	30 android:minWidth="56dp"

	31 android:minHeight="56dp"

	32 android:maxWidth="@dimen/context_menu_header_image_max_size"

	33 android:maxHeight="@dimen/context_menu_header_image_max_size"

	34 android:adjustViewBounds="true"

	35 android:visibility="gone"/>

	36 <TextView

	37 android:id="@+id/context_header_text"

	38 android:layout_width="match_parent"

	39 android:layout_height="wrap_content"

	40 android:gravity="center"

	41 android:maxLines="1"

	42 android:textSize="13sp"

	43 android:textStyle="bold"

	44 android:ellipsize="end"

	45 android:visibility="gone"/>

	46 </LinearLayout>

26 <View	47 <View

27 android:id="@+id/context_divider"	48 android:id="@+id/context_divider"

28 android:layout_width="match_parent"	49 android:layout_width="match_parent"

29 android:layout_height="1dp"	50 android:layout_height="1dp"

30 android:background="#1e000000"/>	51 android:background="#1e000000"/>

31 <ListView	52 <ListView

32 android:id="@+id/selectable_items"	53 android:id="@+id/selectable_items"

33 android:layout_width="match_parent"	54 android:layout_width="match_parent"

34 android:layout_height="wrap_content"	55 android:layout_height="wrap_content"

35 android:paddingTop="8dp"	56 android:paddingTop="8dp"

36 android:paddingBottom="8dp"	57 android:paddingBottom="8dp"

37 android:dividerHeight="0dp"	58 android:dividerHeight="0dp"

38 android:divider="@null"/>	59 android:divider="@null"/>

39 </LinearLayout>	60 </LinearLayout>

OLD	NEW