Open Bug 1372572 Opened 3 years ago Updated Last year

Port MozillaOnline homepage icon code to Fennec

Categories

(Firefox for Android :: General, enhancement, P5)

enhancement

Tracking

()

REOPENED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: mkaply, Assigned: mkaply)

References

(Blocks 1 open bug)

Details

(Whiteboard: [mozillaonline])

Attachments

(2 files)

I'm working to integrate MozillaOnline changes into our mainline Fennec so that they can merge easier (and so that we can see what they are changing).

This is the first patch.

MozillaOnline adds a home button in the URL bar (as a page action). While we want a home button in Fennec, this is not a location where we would do so.

Their original patch is here:

https://github.com/MozillaOnline/gecko-dev/commit/a9ab10aa8438c2b265527583c0529357bc2cf709

I have completely rewritten it and moved the location of the homepage code, but the behavior is the same.
Because I'm using MOZ_ANDROID_MOZILLA_ONLINE in a JS file, I need it in AppConstants.jsm.
Attachment #8877225 - Flags: review?(nalexander)
Comment on attachment 8877225 [details] [diff] [review]
Add MOZ_ANDROID_MOZILLA_ONLINE to AppConstants.jsm

Review of attachment 8877225 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm.
Attachment #8877225 - Flags: review?(nalexander) → review+
Cross-link to Bug 1093113?
Blocks: 1093113
Attachment #8877148 - Flags: review?(snorp) → review?(max)
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/87895cdc8d92
Enable MOZ_ANDROID_MOZILLA_ONLINE for JS files. r=nalexander
https://hg.mozilla.org/mozilla-central/rev/87895cdc8d92
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Comment on attachment 8877148 [details]
Bug 1372572 - Port MozillaOnline code for a homepage page action.

https://reviewboard.mozilla.org/r/148480/#review154482

LGTM
Attachment #8877148 - Flags: review?(max) → review+
Forgot to mark as leaveopen for the rest of the patch.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I'm reasking for review because I discovered a few issues.

1. I needed to add a string for the text of the button because it gets read by Talkback.
2. The button was getting duplicated every time you reopened the browser.
3. I forgot the image for the button. I named it home.png
Mike, this was never marked for re-review: should this be finished or closed?
Flags: needinfo?(mozilla)
Priority: -- → P3
So it ended up getting put on hold because the thought was with the new Quantum stuff on Fennec, it would be easier to add a home button. Do you think it is?
Flags: needinfo?(mozilla)
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.