diff options
author | Danny Baumann <dannybaumann@web.de> | 2016-04-19 11:28:06 +0200 |
---|---|---|
committer | Danny Baumann <dannybaumann@web.de> | 2016-04-25 10:41:08 +0200 |
commit | 3790102575878c57f1fd47260fa03eaa6ff71798 (patch) | |
tree | 7aa2370edd50b19a41bfd9ecf123ab21e6640294 /src/com/android/camera/VideoUI.java | |
parent | 1f7404e8d825878afef2b4c38e30470ac95e9ba5 (diff) | |
download | android_packages_apps_Snap-3790102575878c57f1fd47260fa03eaa6ff71798.tar.gz android_packages_apps_Snap-3790102575878c57f1fd47260fa03eaa6ff71798.tar.bz2 android_packages_apps_Snap-3790102575878c57f1fd47260fa03eaa6ff71798.zip |
Improve RTL layout behaviour.
- Swap top menu items order in RTL
- Swipe in menu from the right in RTL
- Make it possible to select scene/filter items in RTL
- Remove/clean up some nonsense code
Change-Id: I884a06f5cc6e72a1d2ee19dbdf32ce511c8466ec
JIRA:NIGHTLIES-2849
Diffstat (limited to 'src/com/android/camera/VideoUI.java')
-rw-r--r-- | src/com/android/camera/VideoUI.java | 42 |
1 files changed, 17 insertions, 25 deletions
diff --git a/src/com/android/camera/VideoUI.java b/src/com/android/camera/VideoUI.java index 30ecffbad..c28fcca82 100644 --- a/src/com/android/camera/VideoUI.java +++ b/src/com/android/camera/VideoUI.java @@ -697,13 +697,14 @@ public class VideoUI implements PieRenderer.PieListener, } public boolean sendTouchToPreviewMenu(MotionEvent ev) { + ev.offsetLocation(-mPreviewMenuLayout.getLeft(), -mPreviewMenuLayout.getTop()); return mPreviewMenuLayout.dispatchTouchEvent(ev); } public boolean sendTouchToMenu(MotionEvent ev) { if (mMenuLayout != null) { - View v = mMenuLayout.getChildAt(0); - return v.dispatchTouchEvent(ev); + ev.offsetLocation(-mMenuLayout.getLeft(), -mMenuLayout.getTop()); + return mMenuLayout.dispatchTouchEvent(ev); } return false; } @@ -731,7 +732,6 @@ public class VideoUI implements PieRenderer.PieListener, } public void showPopup(ListView popup, int level, boolean animate) { - FrameLayout.LayoutParams layoutParams; hideUI(); popup.setVisibility(View.VISIBLE); @@ -739,16 +739,9 @@ public class VideoUI implements PieRenderer.PieListener, if (mMenuLayout == null) { mMenuLayout = new RotateLayout(mActivity, null); mMenuLayout.setRootView(mRootView); - if (mRootView.getLayoutDirection() != View.LAYOUT_DIRECTION_RTL) { - layoutParams = new FrameLayout.LayoutParams( - CameraActivity.SETTING_LIST_WIDTH_1, LayoutParams.WRAP_CONTENT, - Gravity.LEFT | Gravity.TOP); - } else { - layoutParams = new FrameLayout.LayoutParams( - CameraActivity.SETTING_LIST_WIDTH_1, LayoutParams.WRAP_CONTENT, - Gravity.RIGHT | Gravity.TOP); - } - mMenuLayout.setLayoutParams(layoutParams); + mMenuLayout.setLayoutParams(new FrameLayout.LayoutParams( + CameraActivity.SETTING_LIST_WIDTH_1, LayoutParams.WRAP_CONTENT, + Gravity.START | Gravity.TOP)); mRootView.addView(mMenuLayout); } mMenuLayout.addView(popup); @@ -763,15 +756,6 @@ public class VideoUI implements PieRenderer.PieListener, mRootView.addView(mSubMenuLayout); } - if (mRootView.getLayoutDirection() != View.LAYOUT_DIRECTION_RTL) { - layoutParams = new FrameLayout.LayoutParams( - CameraActivity.SETTING_LIST_WIDTH_2, LayoutParams.WRAP_CONTENT, - Gravity.LEFT | Gravity.TOP); - } else { - layoutParams = new FrameLayout.LayoutParams( - CameraActivity.SETTING_LIST_WIDTH_2, LayoutParams.WRAP_CONTENT, - Gravity.RIGHT | Gravity.TOP); - } final int containerHeight = mRootView.getClientRectForOrientation(mOrientation).height(); @@ -781,10 +765,14 @@ public class VideoUI implements PieRenderer.PieListener, y = Math.max(0, containerHeight - height); } - if (mRootView.getLayoutDirection() != View.LAYOUT_DIRECTION_RTL) { - layoutParams.setMargins(CameraActivity.SETTING_LIST_WIDTH_1, y, 0, 0); - } else { + final FrameLayout.LayoutParams layoutParams = new FrameLayout.LayoutParams( + CameraActivity.SETTING_LIST_WIDTH_2, LayoutParams.WRAP_CONTENT, + Gravity.START | Gravity.TOP); + + if (isRtl()) { layoutParams.setMargins(0, y, CameraActivity.SETTING_LIST_WIDTH_1, 0); + } else { + layoutParams.setMargins(CameraActivity.SETTING_LIST_WIDTH_1, y, 0, 0); } mSubMenuLayout.setLayoutParams(layoutParams); @@ -813,6 +801,10 @@ public class VideoUI implements PieRenderer.PieListener, return mPreviewMenuLayout; } + public boolean isRtl() { + return mRootView.getLayoutDirection() == View.LAYOUT_DIRECTION_RTL; + } + public void showPopup(AbstractSettingPopup popup) { hideUI(); |