The default bug view has changed. See this FAQ.

Status

()

Firefox for Android
General
P1
normal
VERIFIED FIXED
6 years ago
8 months ago

People

(Reporter: elan, Assigned: bnicholson)

Tracking

({feature})

unspecified
ARM
Android
feature
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

(Whiteboard: [birch] [ux needed] [Product Approved)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Text Size

    Default zoom (on page load)

    Open Pages in overview

    Text encoding 

    Block popups

    landscape-only 

    home page

    privacy settings

    enable geolocation

    remember passwords
(Reporter)

Updated

6 years ago
Assignee: nobody → alexp
Whiteboard: [birch]
(Reporter)

Updated

6 years ago
Summary: Preferences- UI Needed → Android Prefs
Whiteboard: [birch] → [birch] [ux needed]
(Assignee)

Updated

6 years ago
Assignee: alexp → bnicholson

Updated

6 years ago
Depends on: 694688
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

6 years ago
Created attachment 569222 [details] [diff] [review]
adds most prefs from XUL fennec

patch requires patch from bug 694688
Attachment #569222 - Flags: review?(mark.finkle)
(Assignee)

Comment 2

6 years ago
Although the homepage can be set in the prefs UI, the browser doesn't actually honor the browser.startup.homepage preference. See bug 696950.
Comment on attachment 569222 [details] [diff] [review]
adds most prefs from XUL fennec

>diff --git a/mobile/chrome/content/browser.js b/mobile/chrome/content/browser.js

>   getPreferences: function getPreferences(aPrefNames) {

>+        switch (prefName) {
>+          case "network.cookie.cookieBehavior":

Add a comment above the switch so we know you are doing specific tweaks to specific preferences. I wonder if there is a better way to handle these. Maybe there isn't. In a XUL app, some of this tweaking would be in the XUL UI code.

>   setPreferences: function setPreferences(aPref) {

>+    switch (json.name) {
>+      case "network.cookie.cookieBehavior":

Same kind of comment here
Attachment #569222 - Flags: review?(mark.finkle) → review+
r+ with nits fixed
(Assignee)

Comment 5

6 years ago
Created attachment 569506 [details] [diff] [review]
rebased and added mfinkle's fixes
Attachment #569222 - Attachment is obsolete: true
https://hg.mozilla.org/projects/birch/rev/2f77b085a9d0
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

6 years ago
Created attachment 569517 [details] [diff] [review]
fixes crash from last patch
Attachment #569517 - Flags: review?(mark.finkle)
Attachment #569517 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/projects/birch/rev/1f61c869b696

pushed followup
Nice!

Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111026 Firefox/10.0a1 Fennec/10.0a1
Status: RESOLVED → VERIFIED
(Reporter)

Updated

6 years ago
Whiteboard: [birch] [ux needed] → [birch] [ux needed] [Product Approved
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
(Reporter)

Updated

5 years ago
Keywords: feature
You need to log in before you can comment on or make changes to this bug.