Skip to content

Commit c482027

Browse files
committed
DeskClock: Show a setting icon instead of single item menu
Change-Id: I1917da96fc983e266a12509468bd4856287136a6
1 parent 6f26c7c commit c482027

4 files changed

Lines changed: 29 additions & 5 deletions

File tree

res/drawable/ic_settings.xml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<!--
3+
SPDX-FileCopyrightText: Material Design Authors / Google LLC
4+
SPDX-License-Identifier: Apache-2.0
5+
-->
6+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
7+
android:width="24dp"
8+
android:height="24dp"
9+
android:viewportWidth="960"
10+
android:viewportHeight="960">
11+
12+
<path
13+
android:fillColor="@android:color/white"
14+
android:pathData="M370,880L354,752Q341,747 329.5,740Q318,733 307,725L188,775L78,585L181,507Q180,500 180,493.5Q180,487 180,480Q180,473 180,466.5Q180,460 181,453L78,375L188,185L307,235Q318,227 330,220Q342,213 354,208L370,80L590,80L606,208Q619,213 630.5,220Q642,227 653,235L772,185L882,375L779,453Q780,460 780,466.5Q780,473 780,480Q780,487 780,493.5Q780,500 778,507L881,585L771,775L653,725Q642,733 630,740Q618,747 606,752L590,880L370,880ZM440,800L519,800L533,694Q564,686 590.5,670.5Q617,655 639,633L738,674L777,606L691,541Q696,527 698,511.5Q700,496 700,480Q700,464 698,448.5Q696,433 691,419L777,354L738,286L639,328Q617,305 590.5,289.5Q564,274 533,266L520,160L441,160L427,266Q396,274 369.5,289.5Q343,305 321,327L222,286L183,354L269,418Q264,433 262,448Q260,463 260,480Q260,496 262,511Q264,526 269,541L183,606L222,674L321,632Q343,655 369.5,670.5Q396,686 427,694L440,800ZM482,620Q540,620 581,579Q622,538 622,480Q622,422 581,381Q540,340 482,340Q423,340 382.5,381Q342,422 342,480Q342,538 382.5,579Q423,620 482,620ZM480,480L480,480Q480,480 480,480Q480,480 480,480L480,480L480,480L480,480Q480,480 480,480Q480,480 480,480Q480,480 480,480Q480,480 480,480L480,480L480,480L480,480Q480,480 480,480Q480,480 480,480L480,480L480,480L480,480Q480,480 480,480Q480,480 480,480L480,480L480,480L480,480Q480,480 480,480Q480,480 480,480Q480,480 480,480Q480,480 480,480L480,480L480,480L480,480Q480,480 480,480Q480,480 480,480L480,480L480,480Z" />
15+
16+
</vector>

src/com/android/deskclock/DeskClock.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ protected void onCreate(Bundle savedInstanceState) {
159159
}
160160

161161
// Configure the menu item controllers add behavior to the toolbar.
162-
mOptionsMenuManager.addMenuItemController(new SettingsMenuItemController(this));
162+
mOptionsMenuManager.addMenuItemController(new SettingsMenuItemController(this, true));
163163

164164
// Inflate the menu during creation to avoid a double layout pass. Otherwise, the menu
165165
// inflation occurs *after* the initial draw and a second layout pass adds in the menu.

src/com/android/deskclock/actionbarmenu/SettingsMenuItemController.java

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,11 @@ public final class SettingsMenuItemController implements MenuItemController {
3636
private static final int SETTING_MENU_RES_ID = R.id.menu_item_settings;
3737

3838
private final Activity mActivity;
39+
private final boolean mShowIcon;
3940

40-
public SettingsMenuItemController(Activity activity) {
41+
public SettingsMenuItemController(Activity activity, boolean showIcon) {
4142
mActivity = activity;
43+
mShowIcon = showIcon;
4244
}
4345

4446
@Override
@@ -48,8 +50,14 @@ public int getId() {
4850

4951
@Override
5052
public void onCreateOptionsItem(Menu menu) {
51-
menu.add(NONE, SETTING_MENU_RES_ID, NONE, R.string.menu_item_settings)
52-
.setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER);
53+
if (mShowIcon) {
54+
menu.add(NONE, SETTING_MENU_RES_ID, NONE, R.string.menu_item_settings)
55+
.setIcon(R.drawable.ic_settings)
56+
.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM);
57+
} else {
58+
menu.add(NONE, SETTING_MENU_RES_ID, NONE, R.string.menu_item_settings)
59+
.setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER);
60+
}
5361
}
5462

5563
@Override

src/com/android/deskclock/worldclock/CitySelectionActivity.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ public boolean onQueryTextChange(String query) {
128128
mOptionsMenuManager.addMenuItemController(new NavUpMenuItemController(this))
129129
.addMenuItemController(mSearchMenuItemController)
130130
.addMenuItemController(new SortOrderMenuItemController())
131-
.addMenuItemController(new SettingsMenuItemController(this));
131+
.addMenuItemController(new SettingsMenuItemController(this, false));
132132
mCitiesList = findViewById(R.id.cities_list);
133133
mCitiesList.setAdapter(mCitiesAdapter);
134134

0 commit comments

Comments
 (0)