Skip to content

Commit eb3acc0

Browse files
Merge pull request #439 from apache/revert-353-patch-1
Revert "Add support for right to left direction languages" I mistakenly merged #353 instead of a similar one.
2 parents 7b16ee8 + 2f74d67 commit eb3acc0

File tree

1 file changed

+10
-17
lines changed

1 file changed

+10
-17
lines changed

src/android/InAppBrowser.java

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,6 @@ Licensed to the Apache Software Foundation (ASF) under one
8585
import java.util.HashMap;
8686
import java.util.StringTokenizer;
8787

88-
import android.content.res.Configuration;
89-
9088
@SuppressLint("SetJavaScriptEnabled")
9189
public class InAppBrowser extends CordovaPlugin {
9290

@@ -747,10 +745,8 @@ private View createCloseButton(int id){
747745
_close = close;
748746
}
749747

750-
Configuration config = activityRes.getConfiguration();
751-
boolean isLeftToRight = config.getLayoutDirection() == View.LAYOUT_DIRECTION_LTR;
752748
RelativeLayout.LayoutParams closeLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
753-
closeLayoutParams.addRule(isLeftToRight ? RelativeLayout.ALIGN_PARENT_RIGHT : RelativeLayout.ALIGN_PARENT_LEFT);
749+
closeLayoutParams.addRule(RelativeLayout.ALIGN_PARENT_RIGHT);
754750
_close.setLayoutParams(closeLayoutParams);
755751

756752
if (Build.VERSION.SDK_INT >= 16)
@@ -777,9 +773,6 @@ public void run() {
777773
dialog.dismiss();
778774
};
779775

780-
Configuration config = cordova.getActivity().getResources().getConfiguration();
781-
boolean isLeftToRight = config.getLayoutDirection() == View.LAYOUT_DIRECTION_LTR;
782-
783776
// Let's create the main dialog
784777
dialog = new InAppBrowserDialog(cordova.getActivity(), android.R.style.Theme_NoTitleBar);
785778
dialog.getWindow().getAttributes().windowAnimations = android.R.style.Animation_Dialog;
@@ -797,25 +790,25 @@ public void run() {
797790
toolbar.setBackgroundColor(toolbarColor);
798791
toolbar.setLayoutParams(new RelativeLayout.LayoutParams(LayoutParams.MATCH_PARENT, this.dpToPixels(44)));
799792
toolbar.setPadding(this.dpToPixels(2), this.dpToPixels(2), this.dpToPixels(2), this.dpToPixels(2));
800-
toolbar.setHorizontalGravity(isLeftToRight ? Gravity.LEFT : Gravity.RIGHT);
793+
toolbar.setHorizontalGravity(Gravity.LEFT);
801794
toolbar.setVerticalGravity(Gravity.TOP);
802795

803796
// Action Button Container layout
804797
RelativeLayout actionButtonContainer = new RelativeLayout(cordova.getActivity());
805798
actionButtonContainer.setLayoutParams(new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT));
806-
actionButtonContainer.setHorizontalGravity(isLeftToRight ? Gravity.LEFT : Gravity.RIGHT);
799+
actionButtonContainer.setHorizontalGravity(Gravity.LEFT);
807800
actionButtonContainer.setVerticalGravity(Gravity.CENTER_VERTICAL);
808801
actionButtonContainer.setId(Integer.valueOf(1));
809802

810803
// Back button
811804
ImageButton back = new ImageButton(cordova.getActivity());
812805
RelativeLayout.LayoutParams backLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
813-
backLayoutParams.addRule(isLeftToRight ? RelativeLayout.ALIGN_LEFT : RelativeLayout.ALIGN_RIGHT);
806+
backLayoutParams.addRule(RelativeLayout.ALIGN_LEFT);
814807
back.setLayoutParams(backLayoutParams);
815808
back.setContentDescription("Back Button");
816809
back.setId(Integer.valueOf(2));
817810
Resources activityRes = cordova.getActivity().getResources();
818-
int backResId = activityRes.getIdentifier(isLeftToRight ? "ic_action_previous_item" : "ic_action_next_item", "drawable", cordova.getActivity().getPackageName());
811+
int backResId = activityRes.getIdentifier("ic_action_previous_item", "drawable", cordova.getActivity().getPackageName());
819812
Drawable backIcon = activityRes.getDrawable(backResId);
820813
if (navigationButtonColor != "") back.setColorFilter(android.graphics.Color.parseColor(navigationButtonColor));
821814
if (Build.VERSION.SDK_INT >= 16)
@@ -837,11 +830,11 @@ public void onClick(View v) {
837830
// Forward button
838831
ImageButton forward = new ImageButton(cordova.getActivity());
839832
RelativeLayout.LayoutParams forwardLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
840-
forwardLayoutParams.addRule(isLeftToRight ? RelativeLayout.RIGHT_OF : RelativeLayout.LEFT_OF, 2);
833+
forwardLayoutParams.addRule(RelativeLayout.RIGHT_OF, 2);
841834
forward.setLayoutParams(forwardLayoutParams);
842835
forward.setContentDescription("Forward Button");
843836
forward.setId(Integer.valueOf(3));
844-
int fwdResId = activityRes.getIdentifier(isLeftToRight ? "ic_action_next_item" : "ic_action_previous_item", "drawable", cordova.getActivity().getPackageName());
837+
int fwdResId = activityRes.getIdentifier("ic_action_next_item", "drawable", cordova.getActivity().getPackageName());
845838
Drawable fwdIcon = activityRes.getDrawable(fwdResId);
846839
if (navigationButtonColor != "") forward.setColorFilter(android.graphics.Color.parseColor(navigationButtonColor));
847840
if (Build.VERSION.SDK_INT >= 16)
@@ -863,8 +856,8 @@ public void onClick(View v) {
863856
// Edit Text Box
864857
edittext = new EditText(cordova.getActivity());
865858
RelativeLayout.LayoutParams textLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.MATCH_PARENT, LayoutParams.MATCH_PARENT);
866-
textLayoutParams.addRule(isLeftToRight ? RelativeLayout.RIGHT_OF : RelativeLayout.LEFT_OF, 1);
867-
textLayoutParams.addRule(isLeftToRight ? RelativeLayout.LEFT_OF : RelativeLayout.RIGHT_OF, 5);
859+
textLayoutParams.addRule(RelativeLayout.RIGHT_OF, 1);
860+
textLayoutParams.addRule(RelativeLayout.LEFT_OF, 5);
868861
edittext.setLayoutParams(textLayoutParams);
869862
edittext.setId(Integer.valueOf(4));
870863
edittext.setSingleLine(true);
@@ -901,7 +894,7 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
901894
footerLayout.addRule(RelativeLayout.ALIGN_PARENT_BOTTOM, RelativeLayout.TRUE);
902895
footer.setLayoutParams(footerLayout);
903896
if (closeButtonCaption != "") footer.setPadding(this.dpToPixels(8), this.dpToPixels(8), this.dpToPixels(8), this.dpToPixels(8));
904-
footer.setHorizontalGravity(isLeftToRight ? Gravity.LEFT : Gravity.RIGHT);
897+
footer.setHorizontalGravity(Gravity.LEFT);
905898
footer.setVerticalGravity(Gravity.BOTTOM);
906899

907900
View footerClose = createCloseButton(7);

0 commit comments

Comments
 (0)