From 1d0b80a25c30ba9da676e915b34236def962a12f Mon Sep 17 00:00:00 2001
From: Cyril Mottier <cyril@cyrilmottier.com>
Date: Thu, 1 Sep 2011 22:33:10 +0200
Subject: [PATCH] Remove the static limit of 3 items in the ActionBar

The limit is now set in the theme. The default value is still 3 as this is considered as the maximum number of global actions a screen SHOULD have. Please do not change this value unless you are completely aware of what you are doing. Modifying it may impair user experience. If you really have a large number of actions, please consider QuickActions.

Closes #36
---
 GreenDroid/res/values/gd_attrs.xml              | 2 ++
 GreenDroid/res/values/gd_styles.xml             | 1 +
 GreenDroid/res/values/gd_themes.xml             | 1 +
 GreenDroid/src/greendroid/widget/ActionBar.java | 6 ++++--
 4 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/GreenDroid/res/values/gd_attrs.xml b/GreenDroid/res/values/gd_attrs.xml
index a91252a..b92a340 100644
--- a/GreenDroid/res/values/gd_attrs.xml
+++ b/GreenDroid/res/values/gd_attrs.xml
@@ -86,6 +86,7 @@
 	<attr name="gdActionBarHomeDrawable" format="reference" />
 	<attr name="gdActionBarItemColorNormal" format="reference|color" />
 	<attr name="gdActionBarItemColorAlt" format="reference|color" />
+	<attr name="gdActionBarMaxItems" format="integer" />
 
 	<attr name="gdActionBarStyle" format="reference" />
 	<attr name="gdActionBarTitleStyle" format="reference" />
@@ -169,6 +170,7 @@
 		<attr name="dividerDrawable" />
 		<attr name="dividerWidth" />
 		<attr name="homeDrawable" format="reference" />
+		<attr name="maxItems" format="integer" />
 	</declare-styleable>
 	
 	<declare-styleable name="AsyncImageView">
diff --git a/GreenDroid/res/values/gd_styles.xml b/GreenDroid/res/values/gd_styles.xml
index 1fea206..8858bfe 100644
--- a/GreenDroid/res/values/gd_styles.xml
+++ b/GreenDroid/res/values/gd_styles.xml
@@ -154,6 +154,7 @@
 		<item name="dividerDrawable">?attr/gdActionBarDividerDrawable</item>
 		<item name="dividerWidth">?attr/gdActionBarDividerWidth</item>
 		<item name="homeDrawable">?attr/gdActionBarHomeDrawable</item>
+		<item name="maxItems">?attr/gdActionBarMaxItems</item>
 	</style>
 	
 	<style name="GreenDroid.Widget.ActionBar.Dashboard">
diff --git a/GreenDroid/res/values/gd_themes.xml b/GreenDroid/res/values/gd_themes.xml
index 831167c..4f4b3f8 100644
--- a/GreenDroid/res/values/gd_themes.xml
+++ b/GreenDroid/res/values/gd_themes.xml
@@ -83,6 +83,7 @@
 		<item name="gdActionBarHomeDrawable">@null</item>
 		<item name="gdActionBarItemColorNormal">@android:color/white</item>
 		<item name="gdActionBarItemColorAlt">@android:color/black</item>
+		<item name="gdActionBarMaxItems">3</item>
 
 		<item name="gdActionBarStyle">@style/GreenDroid.Widget.ActionBar</item>
 		<item name="gdActionBarTitleStyle">@style/GreenDroid.Widget.ActionBar.Title</item>
diff --git a/GreenDroid/src/greendroid/widget/ActionBar.java b/GreenDroid/src/greendroid/widget/ActionBar.java
index b2b0474..f4f07dd 100644
--- a/GreenDroid/src/greendroid/widget/ActionBar.java
+++ b/GreenDroid/src/greendroid/widget/ActionBar.java
@@ -35,7 +35,6 @@
 public class ActionBar extends LinearLayout {
 
     public static final int NONE = 0;
-    private static final int MAX_ITEMS_COUNT = 3;
 
     public enum Type {
         Normal, Dashboard, Empty
@@ -70,6 +69,8 @@ public interface OnActionBarListener {
     private Drawable mDividerDrawable;
     private Drawable mHomeDrawable;
     private int mDividerWidth;
+    
+    private int mMaxItemsCount;
 
     public ActionBar(Context context) {
         this(context, null);
@@ -91,6 +92,7 @@ public ActionBar(Context context, AttributeSet attrs, int defStyle) {
         mDividerDrawable = a.getDrawable(R.styleable.ActionBar_dividerDrawable);
         mDividerWidth = a.getDimensionPixelSize(R.styleable.ActionBar_dividerWidth, -1);
         mHomeDrawable = a.getDrawable(R.styleable.ActionBar_homeDrawable);
+        mMaxItemsCount = a.getInt(R.styleable.ActionBar_maxItems, 3);
         if (mHomeDrawable == null) {
             mHomeDrawable = new ActionBarDrawable(context, R.drawable.gd_action_bar_home);
         }
@@ -185,7 +187,7 @@ public ActionBarItem addItem(ActionBarItem item) {
 
     public ActionBarItem addItem(ActionBarItem item, int itemId) {
 
-        if (mItems.size() >= MAX_ITEMS_COUNT) {
+        if (mItems.size() >= mMaxItemsCount) {
             /*
              * An ActionBar must contain as few items as possible. So let's keep
              * a limit :)