Closed
Bug 865515
Opened 12 years ago
Closed 4 years ago
Rotating multi-pane fragments in Settings displays margins incorrectly
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: liuche, Unassigned)
References
Details
Attachments
(3 files, 2 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•12 years ago
|
||
Was playing around with the new settings, and noticed that the margins are set based on what orientation the tablet is in when Settings is launched, not what orientation the Settings screen is displayed in. This means if Settings is launched in horizontal mode, the larger margin settings persist even when the tablet is rotated to vertical (this cuts off some text); vice versa is also true (with smaller margins persisting).
STR:
1. Have tablet in horizontal mode.
2. Launch Settings - margins are present on either side of the two panes to center the view.
3. Rotate tablet to vertical.
Expected:
Margins decrease to fit smaller horizontal space.
Actual:
Margins remain, even with smaller horizontal viewing space. This cramps the text (and truncates some titles).
Depends on: 773535
Reporter | ||
Comment 2•12 years ago
|
||
Reporter | ||
Comment 3•12 years ago
|
||
Reporter | ||
Comment 5•12 years ago
|
||
Attachment #741612 -
Attachment is obsolete: true
Comment 6•12 years ago
|
||
We probably need to reinflate the content on rotations....
Updated•11 years ago
|
Blocks: new-about-home
Updated•11 years ago
|
No longer blocks: new-about-home
Comment 7•11 years ago
|
||
Does as described above. Pretty straight forward.
Attachment #786732 -
Flags: review?(liuche)
Updated•11 years ago
|
Assignee: nobody → sbhagat
Updated•11 years ago
|
Assignee: sbhagat → nobody
Reporter | ||
Comment 8•11 years ago
|
||
Comment on attachment 786732 [details] [diff] [review]
Patch: Preferences pane now sets correct layout on config change
Clearing the review flag because it turns out that the fix for this would be much more complicated than we first thought.
Removing "onConfigChange" does fix the margin bug, but regresses some other behavior [1]. We decided that there are higher priority bugs to focus on, especially in light of the gains (fixed margins) versus added code complexity of the correct fix.
[1] Removing onConfigChange causes recreation of the activity on all rotations, so we lose state for some dialogs that might be open (Clear private data, Master password, Import from Android, etc). If we ever move to having separate screens for these preferences (unlikely) this would be easier to fix.
The correct fix would be to implement View.onSaveInstanceState and View.onRestoreInstanceState, but to do this correctly involves handling saving some extra preference state. For instance, for Clear private data, there are a series of checkboxes whose state should be saved on rotation, on "confirm", but not on "cancel." One approach would be to save a separate preference for the checkbox state that gets cleared on "cancel" or "commit," but persists through config changes.
Attachment #786732 -
Flags: review?(liuche)
Reporter | ||
Comment 9•11 years ago
|
||
[2] The other regression I forgot to mention is that the Fragment titles disappeared on rotation until a header is clicked.
Comment 10•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
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
•