982190 - Allow addons to specify custom filter 'up' image

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

[Josh Dover [:jdover]]

Addons should be able to specify their own 'up' icon to be used in the filter navigation with an API like:

Home.panels.register("my-panel", {
  title: "My Panel",
  layout: Home.panels.Layout.FRAME,
  views: [{
    type: Home.panels.View.LIST,
    dataset: "my-data",
    backImageUrl: "http://example.com/back-image.png"
  }]
});

Comment 1

[Josh Dover [:jdover]]

Attachment: patch

Addressed comments from bug 975055, comment #41

Comment 2

[Lucas Rocha (:lucasr)]

Comment on attachment 8389336 [details] [diff] [review]
patch

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

Great.

::: mobile/android/base/home/PanelBackItemView.java
@@ +6,5 @@
>  package org.mozilla.gecko.home;
>  
>  import org.mozilla.gecko.R;
>  import org.mozilla.gecko.home.PanelLayout.FilterDetail;
> +import org.mozilla.gecko.home.PanelLayout.FilterManager;

Unused.

Comment 3

[Lucas Rocha (:lucasr)]

I assume this patch doesn't require any change on the JS side? Have you tested this with an actual add-on?

Comment 4

[Josh Dover [:jdover]]

Attachment: patch

(In reply to Lucas Rocha (:lucasr) from comment #3)
> I assume this patch doesn't require any change on the JS side? Have you
> tested this with an actual add-on?

Yes, I did find one bug though where if the backImageUrl isn't specified there was an error with Picasso. I forgot that JSON#optString defaults to an empty string rather than null. Fixed in this patch.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/b4a1adc469bd

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/b4a1adc469bd

Comment 7

[:Margaret Leibovic]

Comment on attachment 8392309 [details] [diff] [review]
patch

Note to release managers: I'm requesting uplift for a series of Firefox Hub bugs. The main fixes we need are in bugs at the end of the series, but trying to rebase those patches proved difficult and risky, so I think we should just uplift the dependecies.

Note to sheriffs: I have a local patch series that I can land on aurora when these bugs all get approval.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): dependency for initial Firefox Hub release (promoted feed add-ons in fx30)
User impact if declined: panels won't update when data changes
Testing completed (on m-c, etc.): baked on m-c
Risk to taking this patch (and alternatives if risky): only affects dynamic panels, need an add-on to trigger this
String or IDL/UUID changes made by this patch: none

Comment 8

[:Margaret Leibovic]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d85cff6f6ade