Closed Bug 773735 Opened 9 years ago Closed 9 years ago

Do not use tablet layout on devices with screen size "large"


(Firefox for Android Graveyard :: Theme and Visual Design, defect)

16 Branch
Not set


(firefox15+ fixed, firefox16 verified)

Firefox 16
Tracking Status
firefox15 + fixed
firefox16 --- verified


(Reporter: mbrubeck, Assigned: mbrubeck)



(Keywords: regression)


(2 files)

In bug 770079 we switched to using the native tablet layout on all devices with screen size "large" or higher.  This includes 7" tablets, and some 5" devices like some versions of the Galaxy Note.  But our tablet UI, at least as currently implemented does not really work well on these mid-size devices, especially in portrait mode.  (See the attached screenshot.)

I don't think the tablet UI is ready for these devices, and at least for Firefox 15 and 16 it would be better to ship the phone UI for "large" screen sizes.  In a later release we can make the UI adapt more specifically to these devices.

For comparison, the XUL Fennec tablet UI was used only on "xlarge" devices (with an exception for "large" Honeycomb tablets, because the XUL phone UI had some problems on those devices).
Attachment #641970 - Attachment is patch: false
Attachment #641970 - Attachment mime type: text/plain → image/png
We can make this change in Native Fennec without re-introducing the complications and duplicate code that bug 770079 removed.

The easiest option is to just rename all of our *-large directories to *-xlarge.  Then we will use the tablet layout only on "xlarge" screens (960x720 "dips" or larger) -- this includes 9" and 10" tablets, and exclude most smaller devices.
Attached patch patchSplinter Review
On IRC, madhava wrote:

> mbrubeck: yeah - based on that definition of large, I wouldn't do tablet UI there
> I know that ibarlow had a different layout in mind for 7" that was more an adapted version of the phone UI
> i.e. tabs at the top

This patch just renames every *-large* path to *-xlarge*, and adjusts the isTablet methods.  (It also takes one step toward combining the two isTablet methods into one.)

For anyone following along:  I know the phone layout is not perfect for 7" tablets; we just think it's better than the tablet layout *for now*.  In future releases we will refine the UI for small tablets.
Assignee: nobody → mbrubeck
Attachment #641986 - Flags: review?(mark.finkle)
Comment on attachment 641986 [details] [diff] [review]

>diff --git a/mobile/android/base/ b/mobile/android/base/

>     public static boolean isTablet() {

>+        return GeckoApp.mAppContext.isTablet();

Given the activity issues and static woes, maybe GeckoAppShell should hold the real impl and GeckoApp can delegate to it.

>diff --git a/mobile/android/base/ b/mobile/android/base/






To minimize confusion, can we rename these variable too? LARGE -> XLARGE


And here
Attachment #641986 - Flags: review?(mark.finkle) → review+
Keywords: uiwanted
Target Milestone: --- → Firefox 16
Comment on attachment 641986 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug  	770079

User impact if declined: Some mid-sized devices switch to the large-screen tablet layout.

Testing completed (on m-c, etc.): Landed on inbound 7/13.

Risk to taking this patch (and alternatives if risky): This is mobile-only and low-risk.  All it does is change "large" to "xlarge" in all of our resource directories.  (So it touches a large number number of files, but the only change to each file is a trivial mechanical change to the filename.)

String or UUID changes made by this patch: None.
Attachment #641986 - Flags: approval-mozilla-aurora?
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #641986 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
We need this mobile-only style change to keep some phablets from getting the tablet UI.
Only able to verify on a Galaxy Tab 2 7" here (1024x600) and I the phone UI.

Verified on Nightly (07/14)
tracking-fennec: ? → ---
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.