Closed
Bug 712477
Opened 13 years ago
Closed 13 years ago
[ICS] Landscape resources are not used in landscape mode
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox11 fixed, firefox12 fixed)
RESOLVED
FIXED
Firefox 12
People
(Reporter: sriram, Assigned: sriram)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mfinkle
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
ICS has separate resources specified for landscape mode. However, they aren't used by Android.
Assignee | ||
Comment 1•13 years ago
|
||
From my primary investigation, if the app is launched in landscape mode (holding the device landscape mode), the landscape specific resources are used. However, they are not used when rotating from portrait mode. This is because the views aren't redrawn during the orientation change.
Comment 2•13 years ago
|
||
(In reply to Sriram Ramasubramanian [:sriram] from comment #1)
> From my primary investigation, if the app is launched in landscape mode
> (holding the device landscape mode), the landscape specific resources are
> used. However, they are not used when rotating from portrait mode. This is
> because the views aren't redrawn during the orientation change.
Is this because we have android:configChanges="orientation" in AndroidManifest.xml?
Assignee | ||
Comment 3•13 years ago
|
||
We need to track "orientation" to avoid restarting activity. However, in onConfigurationChange(), we should redraw the layout for landscape. Which doesn't seem to work.
Assignee | ||
Comment 4•13 years ago
|
||
This patch solves the issue (phew!)
Ideally invalidate() should do the trick. However, it is not taking the new configuration into account while redrawing.
I had to inflate the mBrowserToolbar again. Since we have a different landscape view only for ICS, this is done only for honeycomb+ devices (as the action bar is changed).
The orientation has to be saved to compare against, as inside onConfigurationChanged(), getConfiguration().orientation will be same as newConfig.orientation.
I found few files (drawable-land-v14/ and drawable-port-v14/) not needed. I have removed them in this patch.
Assignee: nobody → sriram
Attachment #583412 -
Flags: review?(mark.finkle)
Updated•13 years ago
|
Attachment #583412 -
Flags: review?(mark.finkle) → review+
Updated•13 years ago
|
Whiteboard: [fennec-aurora]
Comment 5•13 years ago
|
||
Comment 6•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Comment 7•13 years ago
|
||
Comment on attachment 583412 [details] [diff] [review]
Patch
[Approval Request Comment]
User impact if declined: We use the wrong assests in the UI
Testing completed (on m-c, etc.): It's been on m-c for a while
Risk to taking this patch (and alternatives if risky): Low risk
Attachment #583412 -
Flags: approval-mozilla-aurora?
Updated•13 years ago
|
status-firefox11:
--- → affected
status-firefox12:
--- → fixed
Updated•13 years ago
|
Whiteboard: [fennec-aurora]
Comment 8•13 years ago
|
||
Comment on attachment 583412 [details] [diff] [review]
Patch
[Triage Comment]
Mobile only - approved for Aurora.
Attachment #583412 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 9•13 years ago
|
||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•