Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(170)

Unified Diff: content/public/android/java/res/menu/select_action_menu.xml

Issue 2740103006: Implement SmartText selection. (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:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/res/menu/select_action_menu.xml
diff --git a/content/public/android/java/res/menu/select_action_menu.xml b/content/public/android/java/res/menu/select_action_menu.xml
index 30e3e836bd9e70b19111e4bd38f414f7f43e4be1..44854873601c108d967a319e76d9c101d1ce4b8a 100644
--- a/content/public/android/java/res/menu/select_action_menu.xml
+++ b/content/public/android/java/res/menu/select_action_menu.xml
@@ -9,11 +9,20 @@
<!--suppress AlwaysShowAction -->
<menu xmlns:android="http://schemas.android.com/apk/res/android">
<item
+ android:id="@+id/select_action_menu_context_dep"
+ android:title="@android:string/yes"
boliu 2017/03/11 00:37:52 "yes"?
Tima Vaisburd 2017/03/20 05:06:04 string/unknownName to the rescue. The android:titl
+ android:showAsAction="always|withText"
+ android:menuCategory="system"
+ android:orderInCategory="1"
+ />
+ <item
android:id="@+id/select_action_menu_cut"
android:icon="?android:attr/actionModeCutDrawable"
android:title="@android:string/cut"
android:showAsAction="always|withText"
android:alphabeticShortcut="x"
+ android:menuCategory="system"
boliu 2017/03/11 00:37:52 what does do exactly?
+ android:orderInCategory="2"
boliu 2017/03/11 00:37:52 aren't these ordered before? why is it important e
Tima Vaisburd 2017/03/20 05:06:04 The difference is that now I try to add a menu ite
/>
<item
android:id="@+id/select_action_menu_copy"
@@ -21,6 +30,8 @@
android:title="@android:string/copy"
android:showAsAction="always|withText"
android:alphabeticShortcut="c"
+ android:menuCategory="system"
+ android:orderInCategory="3"
/>
<item
android:id="@+id/select_action_menu_paste"
@@ -28,12 +39,16 @@
android:title="@android:string/paste"
android:showAsAction="always|withText"
android:alphabeticShortcut="v"
+ android:menuCategory="system"
+ android:orderInCategory="4"
/>
<item
android:id="@+id/select_action_menu_share"
android:title="@string/actionbar_share"
android:showAsAction="always|withText"
style="@style/SelectActionMenuShare"
+ android:menuCategory="system"
+ android:orderInCategory="5"
/>
<item
android:id="@+id/select_action_menu_select_all"
@@ -41,14 +56,20 @@
android:title="@android:string/selectAll"
android:showAsAction="always|withText"
android:alphabeticShortcut="a"
+ android:menuCategory="system"
+ android:orderInCategory="6"
/>
<group
- android:id="@+id/select_action_menu_text_processing_menus">
+ android:id="@+id/select_action_menu_text_processing_menus"
+ android:menuCategory="system"
+ android:orderInCategory="7">
</group>
<item
android:id="@+id/select_action_menu_web_search"
android:title="@string/actionbar_web_search"
android:showAsAction="always|withText"
style="@style/SelectActionMenuWebSearch"
+ android:menuCategory="system"
+ android:orderInCategory="8"
/>
</menu>

Powered by Google App Engine
This is Rietveld 408576698