OSDN Git Service

Fix several UI issues.
authorOwen Lin <owenlin@google.com>
Mon, 15 Mar 2010 06:08:13 +0000 (14:08 +0800)
committerOwen Lin <owenlin@google.com>
Tue, 16 Mar 2010 06:20:33 +0000 (14:20 +0800)
1. Remove the cross in border_last_picture
2. Remove the gripper in VideoCamera
3. Solve the popup triangle locate in wrong position
4. Remove the top and bottom borders of the preview frame in Camera
5. Rotate the on-screen indicators when orientation changed
6. Add the checking for the flash mode in VideoCamera

Change-Id: I304ea3582d08c5e46ed750b7371588635a4d8009

22 files changed:
res/drawable-hdpi/border_last_picture_highlight.9.png
res/drawable-hdpi/border_last_picture_normal.9.png
res/drawable-hdpi/border_last_picture_pressed.9.png
res/drawable-hdpi/border_view_finder.9.png
res/drawable-hdpi/gripper_handle.9.png [deleted file]
res/drawable-mdpi/border_last_picture_highlight.9.png
res/drawable-mdpi/border_last_picture_normal.9.png
res/drawable-mdpi/border_last_picture_pressed.9.png
res/drawable-mdpi/gripper_handle.9.png [deleted file]
res/drawable-mdpi/menu_popup.9.png
res/drawable-mdpi/menu_popup_triangle.png
res/layout/on_screen_menu.xml [deleted file]
res/layout/on_screen_menu_header.xml [deleted file]
res/layout/on_screen_menu_list_item.xml [deleted file]
res/layout/video_camera.xml
src/com/android/camera/CameraSettings.java
src/com/android/camera/PreviewFrameLayout.java
src/com/android/camera/RotateImageView.java
src/com/android/camera/ui/AbstractIndicator.java
src/com/android/camera/ui/GLRootView.java
src/com/android/camera/ui/HeadUpDisplay.java
src/com/android/camera/ui/IndicatorBar.java

index d92e795..ae4a0e2 100644 (file)
Binary files a/res/drawable-hdpi/border_last_picture_highlight.9.png and b/res/drawable-hdpi/border_last_picture_highlight.9.png differ
index 4ed2a34..64d17a0 100644 (file)
Binary files a/res/drawable-hdpi/border_last_picture_normal.9.png and b/res/drawable-hdpi/border_last_picture_normal.9.png differ
index bcdb34b..2c7ba22 100644 (file)
Binary files a/res/drawable-hdpi/border_last_picture_pressed.9.png and b/res/drawable-hdpi/border_last_picture_pressed.9.png differ
index 07737df..b5922bf 100644 (file)
Binary files a/res/drawable-hdpi/border_view_finder.9.png and b/res/drawable-hdpi/border_view_finder.9.png differ
diff --git a/res/drawable-hdpi/gripper_handle.9.png b/res/drawable-hdpi/gripper_handle.9.png
deleted file mode 100644 (file)
index e7a2c20..0000000
Binary files a/res/drawable-hdpi/gripper_handle.9.png and /dev/null differ
index 8bbac44..a78cd12 100755 (executable)
Binary files a/res/drawable-mdpi/border_last_picture_highlight.9.png and b/res/drawable-mdpi/border_last_picture_highlight.9.png differ
index 0bebf8f..021286f 100755 (executable)
Binary files a/res/drawable-mdpi/border_last_picture_normal.9.png and b/res/drawable-mdpi/border_last_picture_normal.9.png differ
index 7671494..d18e929 100755 (executable)
Binary files a/res/drawable-mdpi/border_last_picture_pressed.9.png and b/res/drawable-mdpi/border_last_picture_pressed.9.png differ
diff --git a/res/drawable-mdpi/gripper_handle.9.png b/res/drawable-mdpi/gripper_handle.9.png
deleted file mode 100644 (file)
index b599583..0000000
Binary files a/res/drawable-mdpi/gripper_handle.9.png and /dev/null differ
index a655a03..2a75324 100644 (file)
Binary files a/res/drawable-mdpi/menu_popup.9.png and b/res/drawable-mdpi/menu_popup.9.png differ
index 91b6523..52b805e 100644 (file)
Binary files a/res/drawable-mdpi/menu_popup_triangle.png and b/res/drawable-mdpi/menu_popup_triangle.png differ
diff --git a/res/layout/on_screen_menu.xml b/res/layout/on_screen_menu.xml
deleted file mode 100644 (file)
index f80a846..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Copyright (C) 2007 The Android Open Source Project
-
-     Licensed under the Apache License, Version 2.0 (the "License");
-     you may not use this file except in compliance with the License.
-     You may obtain a copy of the License at
-
-          http://www.apache.org/licenses/LICENSE-2.0
-
-     Unless required by applicable law or agreed to in writing, software
-     distributed under the License is distributed on an "AS IS" BASIS,
-     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-     See the License for the specific language governing permissions and
-     limitations under the License.
--->
-
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-        android:id="@+id/main_panel"
-        android:background="@drawable/camera_background"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content">
-    <FrameLayout android:layout_width="match_parent"
-            android:background="@android:color/black"
-            android:layout_height="match_parent"
-            android:layout_toLeftOf="@+id/btn_gripper">
-        <ListView android:id="@+id/menu_view"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent" />
-        <ListView android:id="@+id/sub_menu"
-                android:visibility="invisible"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent" />
-    </FrameLayout>
-    <ImageView android:id="@+id/btn_gripper"
-            android:layout_height="match_parent"
-            android:layout_width="wrap_content"
-            android:background="@drawable/gripper_handle"
-            android:layout_alignParentRight="true"/>
-</RelativeLayout>
diff --git a/res/layout/on_screen_menu_header.xml b/res/layout/on_screen_menu_header.xml
deleted file mode 100644 (file)
index 29e9176..0000000
+++ /dev/null
@@ -1,24 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Copyright (C) 2009 The Android Open Source Project
-
-     Licensed under the Apache License, Version 2.0 (the "License");
-     you may not use this file except in compliance with the License.
-     You may obtain a copy of the License at
-
-          http://www.apache.org/licenses/LICENSE-2.0
-
-     Unless required by applicable law or agreed to in writing, software
-     distributed under the License is distributed on an "AS IS" BASIS,
-     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-     See the License for the specific language governing permissions and
-     limitations under the License.
--->
-<TextView xmlns:android="http://schemas.android.com/apk/res/android"
-        android:id="@+id/title"
-        android:background="#222"
-        android:paddingLeft="5dp"
-        android:paddingRight="5dp"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:textAppearance="?android:attr/textAppearanceSmall"
-        android:singleLine="true" />
diff --git a/res/layout/on_screen_menu_list_item.xml b/res/layout/on_screen_menu_list_item.xml
deleted file mode 100644 (file)
index de83a4c..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Copyright (C) 2009 The Android Open Source Project
-
-     Licensed under the Apache License, Version 2.0 (the "License");
-     you may not use this file except in compliance with the License.
-     You may obtain a copy of the License at
-
-          http://www.apache.org/licenses/LICENSE-2.0
-
-     Unless required by applicable law or agreed to in writing, software
-     distributed under the License is distributed on an "AS IS" BASIS,
-     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-     See the License for the specific language governing permissions and
-     limitations under the License.
--->
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_weight="1"
-        android:gravity="center_vertical"
-        android:minHeight="?android:attr/listPreferredItemHeight"
-        android:paddingLeft="10dp"
-        android:paddingRight="5dp" >
-    <TextView android:id="@+id/title"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:maxLines="2"
-            android:ellipsize="end"
-            android:textAppearance="?android:attr/textAppearanceLarge"
-            android:fadingEdge="horizontal" />
-
-    <TextView android:id="@+id/summary"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:layout_below="@id/title"
-            android:layout_alignLeft="@id/title"
-            android:textAppearance="?android:attr/textAppearanceSmall"
-            android:singleLine="true" />
-</RelativeLayout>
index 482750c..3172f28 100644 (file)
             android:layout_width="match_parent"
             android:layout_height="match_parent"
             android:layout_weight="1">
-        <ImageView android:id="@+id/btn_gripper"
-                android:background="@drawable/btn_gripper"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"/>
         <FrameLayout android:id="@+id/frame"
                 android:layout_width="match_parent"
                 android:layout_height="match_parent"
index 774bc5a..1e7cec2 100644 (file)
@@ -26,8 +26,6 @@ import android.media.CamcorderProfile;
 import android.preference.PreferenceManager;
 import android.util.Log;
 
-import com.android.camera.R;
-
 import java.util.ArrayList;
 import java.util.List;
 
@@ -138,6 +136,8 @@ public class CameraSettings {
         ListPreference flashMode = group.findPreference(KEY_FLASH_MODE);
         ListPreference focusMode = group.findPreference(KEY_FOCUS_MODE);
         ListPreference exposure = group.findPreference(KEY_EXPOSURE);
+        ListPreference videoFlashMode =
+                group.findPreference(KEY_VIDEOCAMERA_FLASH_MODE);
 
         // Since the screen could be loaded from different resources, we need
         // to check if the preference is available here
@@ -181,6 +181,10 @@ public class CameraSettings {
             filterUnsupportedOptions(group,
                     focusMode, mParameters.getSupportedFocusModes());
         }
+        if (videoFlashMode != null) {
+            filterUnsupportedOptions(group,
+                    videoFlashMode, mParameters.getSupportedFlashModes());
+        }
 
         if (exposure != null) {
             buildExposureCompensation(group, exposure);
index 09081bb..d30f6c8 100644 (file)
@@ -20,13 +20,9 @@ import android.app.Activity;
 import android.content.Context;
 import android.util.AttributeSet;
 import android.util.DisplayMetrics;
-import android.view.View;
 import android.view.ViewGroup;
-import android.widget.ImageView;
 import android.widget.FrameLayout;
 
-import com.android.camera.R;
-
 /**
  * A layout which handles the preview aspect ratio and the position of
  * the gripper.
@@ -40,10 +36,12 @@ public class PreviewFrameLayout extends ViewGroup {
     }
 
     private double mAspectRatio = 4.0 / 3.0;
-    private ImageView mGripper;
     private FrameLayout mFrame;
     private OnSizeChangedListener mSizeListener;
-    private DisplayMetrics mMetrics = new DisplayMetrics();
+    private final DisplayMetrics mMetrics = new DisplayMetrics();
+
+    private int mPreviewWidth;
+    private int mPreviewHeight;
 
     public PreviewFrameLayout(Context context, AttributeSet attrs) {
         super(context, attrs);
@@ -57,7 +55,6 @@ public class PreviewFrameLayout extends ViewGroup {
 
     @Override
     protected void onFinishInflate() {
-        mGripper = (ImageView) findViewById(R.id.btn_gripper);
         mFrame = (FrameLayout) findViewById(R.id.frame);
         if (mFrame == null) {
             throw new IllegalStateException(
@@ -75,29 +72,26 @@ public class PreviewFrameLayout extends ViewGroup {
     }
 
     @Override
-    protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
-        super.onMeasure(widthMeasureSpec, heightMeasureSpec);
-
-        int gripperWidth = 0;
-        int gripperHeight = 0;
-
-        if (mGripper != null) {
-            measureChild(mGripper, widthMeasureSpec, heightMeasureSpec);
-            gripperWidth = mGripper.getMeasuredWidth();
-            gripperHeight = mGripper.getMeasuredHeight();
-        }
+    protected void onLayout(boolean changed, int l, int t, int r, int b) {
+        // Try to layout the "frame" in the center of the area, and put
+        // "gripper" just to the left of it. If there is no enough space for
+        // the gripper, the "frame" will be moved a little right so that
+        // they won't overlap with each other.
 
-        int frameWidth = getMeasuredWidth() - (int) Math.max(
-                gripperWidth, MIN_HORIZONTAL_MARGIN * mMetrics.density);
-        int frameHeight = getMeasuredHeight();
+        int frameWidth = getWidth();
+        int frameHeight = getHeight();
 
         FrameLayout f = mFrame;
 
-        int horizontalPadding = f.getPaddingLeft() + f.getPaddingRight();
+        int horizontalPadding = Math.max(
+                f.getPaddingLeft() + f.getPaddingRight(),
+                (int) (MIN_HORIZONTAL_MARGIN * mMetrics.density));
         int verticalPadding = f.getPaddingBottom() + f.getPaddingTop();
 
+        // Ignore the vertical paddings, so that we won't draw the frame on the
+        // top and bottom sides
+        int previewHeight = frameHeight;
         int previewWidth = frameWidth - horizontalPadding;
-        int previewHeight = frameHeight - verticalPadding;
 
         // resize frame and preview for aspect ratio
         if (previewWidth > previewHeight * mAspectRatio) {
@@ -105,46 +99,19 @@ public class PreviewFrameLayout extends ViewGroup {
         } else {
             previewHeight = (int) (previewWidth / mAspectRatio + .5);
         }
+
         frameWidth = previewWidth + horizontalPadding;
         frameHeight = previewHeight + verticalPadding;
 
-        measureChild(mFrame,
-                MeasureSpec.makeMeasureSpec(MeasureSpec.EXACTLY, frameWidth),
-                MeasureSpec.makeMeasureSpec(MeasureSpec.EXACTLY, frameHeight));
-    }
-
-    @Override
-    protected void onLayout(boolean changed, int l, int t, int r, int b) {
-        // Try to layout the "frame" in the center of the area, and put
-        // "gripper" just to the left of it. If there is no enough space for
-        // the gripper, the "frame" will be moved a little right so that
-        // they won't overlap with each other.
-
-        int frameWidth = mFrame.getMeasuredWidth();
-        int frameHeight = mFrame.getMeasuredHeight();
-
-        int leftSpace = ((r - l) - frameWidth) / 2;
-        int topSpace = ((b - t) - frameHeight) / 2;
-
-        int gripperWidth = 0;
-        int gripperHeight = 0;
-        if (mGripper != null) {
-            gripperWidth = mGripper.getMeasuredWidth();
-            gripperHeight = mGripper.getMeasuredHeight();
-            myLayoutChild(mGripper,
-                    Math.max(l, l + (leftSpace - gripperWidth)),
-                    t + ((b - t) - gripperHeight) / 2,
-                    gripperWidth, gripperHeight);
-        }
-        myLayoutChild(mFrame, Math.max(l + leftSpace, l + gripperWidth),
-                t + topSpace, frameWidth, frameHeight);
+        int hSpace = ((r - l) - frameWidth) / 2;
+        int vSpace = ((b - t) - frameHeight) / 2;
+        mFrame.measure(
+                MeasureSpec.makeMeasureSpec(frameWidth, MeasureSpec.EXACTLY),
+                MeasureSpec.makeMeasureSpec(frameHeight, MeasureSpec.EXACTLY));
+        mFrame.layout(l + hSpace, t + vSpace, r - hSpace, b - vSpace);
         if (mSizeListener != null) {
             mSizeListener.onSizeChanged();
         }
     }
-
-    private static void myLayoutChild(View child, int l, int t, int w, int h) {
-        child.layout(l, t, l + w, t + h);
-    }
 }
 
index 14d1628..dcec5e4 100644 (file)
@@ -21,7 +21,6 @@ import android.graphics.Canvas;
 import android.graphics.Rect;
 import android.graphics.drawable.Drawable;
 import android.util.AttributeSet;
-import android.view.animation.AnimationUtils;
 import android.widget.ImageView;
 
 /**
@@ -32,16 +31,7 @@ public class RotateImageView extends ImageView {
     @SuppressWarnings("unused")
     private static final String TAG = "RotateImageView";
 
-    private static final int ANIMATION_SPEED = 180; // 180 deg/sec
-
-    private int mCurrentDegree = 0; // [0, 359]
-    private int mStartDegree = 0;
-    private int mTargetDegree = 0;
-
-    private boolean mClockwise = false;
-
-    private long mAnimationStartTime = 0;
-    private long mAnimationEndTime = 0;
+    private int mDegree = 0; // [0, 359]
 
     public RotateImageView(Context context, AttributeSet attrs) {
         super(context, attrs);
@@ -50,23 +40,8 @@ public class RotateImageView extends ImageView {
     public void setDegree(int degree) {
         // make sure in the range of [0, 359]
         degree = degree >= 0 ? degree % 360 : degree % 360 + 360;
-        if (degree == mTargetDegree) return;
-
-        mTargetDegree = degree;
-        mStartDegree = mCurrentDegree;
-        mAnimationStartTime = AnimationUtils.currentAnimationTimeMillis();
-
-        int diff = mTargetDegree - mCurrentDegree;
-        diff = diff >= 0 ? diff : 360 + diff; // make it in range [0, 359]
-
-        // Make it in range [-179, 180]. That's the shorted distance between the
-        // two angles
-        diff = diff > 180 ? diff - 360 : diff;
-
-        mClockwise = diff >= 0;
-        mAnimationEndTime = mAnimationStartTime
-                + Math.abs(diff) * 1000 / ANIMATION_SPEED;
-
+        if (degree == mDegree) return;
+        mDegree = degree;
         invalidate();
     }
 
@@ -74,31 +49,27 @@ public class RotateImageView extends ImageView {
     protected void onDraw(Canvas canvas) {
 
         Drawable drawable = getDrawable();
-        if (drawable == null) return;
 
+        if (drawable == null) return;
         Rect bounds = drawable.getBounds();
         int w = bounds.right - bounds.left;
         int h = bounds.bottom - bounds.top;
 
         if (w == 0 || h == 0) return; // nothing to draw
 
-        if (mCurrentDegree != mTargetDegree) {
-            long time = AnimationUtils.currentAnimationTimeMillis();
-            if (time < mAnimationEndTime) {
-                int deltaTime = (int)(time - mAnimationStartTime);
-                int degree = mStartDegree + ANIMATION_SPEED
-                        * (mClockwise ? deltaTime : -deltaTime) / 1000;
-                degree = degree >= 0 ? degree % 360 : degree % 360 + 360;
-                mCurrentDegree = degree;
-                invalidate();
-            } else {
-                mCurrentDegree = mTargetDegree;
-            }
-        }
-
         int saveCount = canvas.getSaveCount();
-        canvas.translate(getPaddingLeft(), getPaddingTop());
-        canvas.rotate(-mCurrentDegree, w / 2, h / 2);
+
+        int left = getPaddingLeft();
+        int top = getPaddingTop();
+        int right = getPaddingRight();
+        int bottom = getPaddingBottom();
+
+        int width = getWidth() - left - right;
+        int height = getHeight() - top - bottom;
+
+        canvas.translate(left + width / 2, top + height / 2);
+        canvas.rotate(-mDegree);
+        canvas.translate(-w / 2, -h / 2);
         drawable.draw(canvas);
         canvas.restoreToCount(saveCount);
     }
index 7b93205..7c6df7c 100644 (file)
@@ -2,12 +2,16 @@
 package com.android.camera.ui;
 
 import android.content.Context;
+import android.graphics.Matrix;
 import android.graphics.Rect;
+import android.view.animation.AlphaAnimation;
+import android.view.animation.Transformation;
 
 import javax.microedition.khronos.opengles.GL11;
 
 public abstract class AbstractIndicator extends GLView {
     private static final int DEFAULT_PADDING = 3;
+    private int mOrientation = 0;
 
     abstract protected Texture getIcon();
 
@@ -27,18 +31,37 @@ public abstract class AbstractIndicator extends GLView {
     @Override
     protected void render(GLRootView root, GL11 gl) {
         Texture icon = getIcon();
-
-        Rect p = mPaddings;
-        int width = getWidth() - p.left - p.right;
-        int height = getHeight() - p.top - p.bottom;
-
         if (icon != null) {
-            icon.draw(root,
-                    p.left + (width - icon.getWidth()) / 2,
-                    p.top + (height - icon.getHeight()) / 2);
+            Rect p = mPaddings;
+            int width = getWidth() - p.left - p.right;
+            int height = getHeight() - p.top - p.bottom;
+            if (mOrientation != 0) {
+                Transformation trans = root.pushTransform();
+                Matrix matrix = trans.getMatrix();
+                matrix.preTranslate(p.left + width / 2, p.top + height / 2);
+                matrix.preRotate(-mOrientation);
+                icon.draw(root, -icon.getWidth() / 2, -icon.getHeight() / 2);
+                root.popTransform();
+            } else {
+                icon.draw(root,
+                        p.left + (width - icon.getWidth()) / 2,
+                        p.top + (height - icon.getHeight()) / 2);
+            }
         }
     }
 
+    public void setOrientation(int orientation) {
+        if (orientation % 90 != 0) throw new IllegalArgumentException();
+        orientation = orientation % 360;
+        if (orientation < 0) orientation += 360;
+
+        if (mOrientation == orientation) return;
+        mOrientation = orientation;
+        AlphaAnimation anim = new AlphaAnimation(0.2f, 1);
+        anim.setDuration(200);
+        startAnimation(anim);
+    }
+
     abstract public GLView getPopupContent();
 
     abstract public void overrideSettings(String key, String settings);
index 84eea06..92c0f35 100644 (file)
@@ -118,10 +118,11 @@ public class GLRootView extends GLSurfaceView
         return mTransformation;
     }
 
-    public void pushTransform() {
+    public Transformation pushTransform() {
         Transformation trans = obtainTransformation();
         trans.set(mTransformation);
         mTransformStack.push(trans);
+        return mTransformation;
     }
 
     public void popTransform() {
@@ -400,14 +401,14 @@ public class GLRootView extends GLSurfaceView
         }
 
         if ((mFlags & FLAG_NEED_LAYOUT) != 0) layoutContentPane();
-
+        clearClip();
         gl.glClear(GL10.GL_COLOR_BUFFER_BIT);
         gl.glEnable(GL11.GL_BLEND);
         gl.glBlendFunc(GL11.GL_ONE, GL11.GL_ONE_MINUS_SRC_ALPHA);
 
         /*gl.glDisable(GL11.GL_TEXTURE_2D);
         gl.glColor4f(0, 0, 0.5f, 0.4f);
-        drawRect(30, 30, 30, 30);
+        drawRect(0, 0, 725, 480);
         gl.glEnable(GL11.GL_TEXTURE_2D);*/
 
         mAnimationTime = SystemClock.uptimeMillis();
index fc2628e..561bd5b 100644 (file)
@@ -17,12 +17,11 @@ import android.view.View.MeasureSpec;
 import android.view.animation.AlphaAnimation;
 import android.view.animation.Animation;
 
-import com.android.camera.R;
-
 import com.android.camera.CameraSettings;
 import com.android.camera.IconListPreference;
 import com.android.camera.ListPreference;
 import com.android.camera.PreferenceGroup;
+import com.android.camera.R;
 
 import java.util.ArrayList;
 import java.util.concurrent.Callable;
@@ -58,8 +57,7 @@ public class HeadUpDisplay extends GLView {
 
     private PopupWindow mPopupWindow;
 
-    private int mAnchorX;
-    private int mAnchorY;
+    private GLView mAnchorView;
     private int mOrientation = 0;
 
     protected Listener mListener;
@@ -178,7 +176,7 @@ public class HeadUpDisplay extends GLView {
 
         if(mPopupWindow != null
                 && mPopupWindow.getVisibility() == GLView.VISIBLE) {
-            layoutPopupWindow(mAnchorX, mAnchorY);
+            layoutPopupWindow(mAnchorView);
         }
     }
 
@@ -190,9 +188,14 @@ public class HeadUpDisplay extends GLView {
         initializeIndicatorBar(context, preferenceGroup);
     }
 
-    private void layoutPopupWindow(int anchorX, int anchorY) {
-        mAnchorX = anchorX;
-        mAnchorY = anchorY;
+    private void layoutPopupWindow(GLView anchorView) {
+
+        mAnchorView = anchorView;
+        Rect rect = new Rect();
+        getBoundsOf(anchorView, rect);
+
+        int anchorX = rect.left + sPopupWindowOverlap;
+        int anchorY = (rect.top + rect.bottom) / 2;
 
         int width = (int) (getWidth() * MAX_WIDTH_RATIO + .5);
         int height = (int) (getHeight() * MAX_HEIGHT_RATIO + .5);
@@ -215,8 +218,8 @@ public class HeadUpDisplay extends GLView {
                 xoffset, yoffset, xoffset + width, yoffset + height);
     }
 
-    private void showPopupWindow(int anchorX, int anchorY) {
-        layoutPopupWindow(anchorX, anchorY);
+    private void showPopupWindow(GLView anchorView) {
+        layoutPopupWindow(anchorView);
         mPopupWindow.popup();
         mSharedPrefs.registerOnSharedPreferenceChangeListener(
                 mSharedPreferenceChangeListener);
@@ -248,9 +251,10 @@ public class HeadUpDisplay extends GLView {
 
     public void setOrientation(int orientation) {
         mOrientation = orientation;
+        mIndicatorBar.setOrientation(orientation);
         if (mPopupWindow == null) return;
         if (mPopupWindow.getVisibility() == GLView.VISIBLE) {
-            Animation alpha = new AlphaAnimation(0, 1);
+            Animation alpha = new AlphaAnimation(0.2f, 1);
             alpha.setDuration(250);
             mPopupWindow.startAnimation(alpha);
             scheduleDeactiviateIndicatorBar();
@@ -333,10 +337,6 @@ public class HeadUpDisplay extends GLView {
             implements IndicatorBar.OnItemSelectedListener {
 
         public void onItemSelected(GLView view, int position) {
-            Rect rect = new Rect();
-            getBoundsOf(view, rect);
-            int anchorX = rect.left + sPopupWindowOverlap;
-            int anchorY = (rect.top + rect.bottom) / 2;
 
             AbstractIndicator indicator = (AbstractIndicator) view;
             if (mPopupWindow == null) {
@@ -345,9 +345,9 @@ public class HeadUpDisplay extends GLView {
             mPopupWindow.setContent(indicator.getPopupContent());
 
             if (mPopupWindow.getVisibility() == GLView.VISIBLE) {
-                layoutPopupWindow(anchorX, anchorY);
+                layoutPopupWindow(indicator);
             } else {
-                showPopupWindow(anchorX, anchorY);
+                showPopupWindow(indicator);
             }
         }
 
index 5ea9261..090a71a 100644 (file)
@@ -175,4 +175,10 @@ public class IndicatorBar extends GLView {
             ((AbstractIndicator) getComponent(i)).reloadPreferences();
         }
     }
+
+    public void setOrientation(int orientation) {
+        for (int i = 1, n = getComponentCount(); i < n; ++i) {
+            ((AbstractIndicator) getComponent(i)).setOrientation(orientation);
+        }
+    }
 }