Skip to content

Commit e82bb1d

Browse files
committed
Merge branch 'hotfix/3.1.1' into develop
2 parents 0df2983 + d26ad1e commit e82bb1d

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

WordPress/src/main/java/org/wordpress/android/ui/themes/ThemeDetailsFragment.java

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -292,27 +292,24 @@ private void loadFeatureView(ArrayList<String> featuresArray) {
292292
* @param context
293293
* @author Karim Varela
294294
**/
295-
private void populateViews(LinearLayout linearLayout, View[] views, Context context)
296-
{
297-
RelativeLayout.LayoutParams llParams = (android.widget.RelativeLayout.LayoutParams) linearLayout.getLayoutParams();
295+
private void populateViews(LinearLayout linearLayout, View[] views, Context context) {
296+
RelativeLayout.LayoutParams llParams =
297+
(android.widget.RelativeLayout.LayoutParams) linearLayout.getLayoutParams();
298298

299299
Display display = getActivity().getWindowManager().getDefaultDisplay();
300300
linearLayout.removeAllViews();
301301

302-
int maxWidth = display.getWidth() - llParams.leftMargin - llParams.rightMargin - mParentView.getPaddingLeft() - mParentView.getPaddingRight();
303-
302+
int maxWidth = display.getWidth() - llParams.leftMargin - llParams.rightMargin - mParentView.getPaddingLeft()
303+
- mParentView.getPaddingRight();
304304

305305
if (DisplayUtils.isXLarge(getActivity())) {
306306
int minDialogWidth = getResources().getDimensionPixelSize(R.dimen.theme_details_dialog_min_width);
307307
int dialogWidth = Math.max((int) (display.getWidth() * 0.6), minDialogWidth);
308308
maxWidth = dialogWidth / 2 - llParams.leftMargin - llParams.rightMargin;
309-
310-
} else if (DisplayUtils.isTablet(getActivity()) && mLeftContainer != null) {
309+
} else if (mLeftContainer != null && mLeftContainer.getLayoutParams() instanceof LinearLayout.LayoutParams) {
311310
int spec = MeasureSpec.makeMeasureSpec(0, MeasureSpec.UNSPECIFIED);
312311
mLeftContainer.measure(spec, spec);
313-
314312
LinearLayout.LayoutParams params = (LayoutParams) mLeftContainer.getLayoutParams();
315-
316313
maxWidth -= mLeftContainer.getMeasuredWidth() + params.rightMargin + params.leftMargin;
317314
}
318315

libs/utils/WordPressUtils/src/main/java/org/wordpress/android/util/DisplayUtils.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,10 @@ public static int pxToDp(Context context, int px) {
5252
return (int) ((px/displayMetrics.density)+0.5);
5353
}
5454

55+
/**
56+
* Deprecated method, returns true on some phones.
57+
*/
58+
@Deprecated
5559
public static boolean isTablet(Context context) {
5660
// http://stackoverflow.com/a/8427523/1673548
5761
if (context == null)

0 commit comments

Comments
 (0)