Closed Bug 968573 Opened 6 years ago Closed 6 years ago

Refine interaction of clicks on dynamic panel items

Categories

(Firefox for Android :: Awesomescreen, defect)

ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: Margaret, Assigned: lucasr)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

I think we should find a way to default to opening regular URLs in the fennec you're in, instead of offering (or automatically) opening the URL in a different browser.

Additionally, we should just open that URL in the tab you're in, instead of opening it in a new tab. Basically, I think we should match the behavior of our built-in panels.
(In reply to :Margaret Leibovic from comment #0)
> I think we should find a way to default to opening regular URLs in the
> fennec you're in, instead of offering (or automatically) opening the URL in
> a different browser.

This is not necessarily going to be opened in a "browser" btw. It all comes down to the intent URL patterns that the apps you have in your device are tracking. For example, the Twitter app tracks http://twitter.com/* URLs, YouTube tracks http://youtube.com/* URLs, and so on.

IIRC, our plan for that was to have a panel view option to toggle between 'intent' or 'browser'. Something along the lines of:

ItemHandler {
  INTENT: "intent",
  BROWSER: "browser"
}

{ type: Home.panels.View.LIST,
  dataset: "somedatasetid",
  itemHandler: Home.panels.ItemHandler.INTENT }

The default value being Home.panels.ItemHandler.BROWSER.

Thoughts?
That sounds good, I didn't see a bug filed about this plan, so we can use this one.

I agree the default should just be to open the URL in the browser, and I think the behavior here should be to open it in the current tab.

Allowing the add-on to launch an intent is a nice option, but I think that should be deliberately specified by the add-on. The API you proposed seems reasonable to me.
Probably out of scope, but there are these intent uri things in Android. I started to support them at one point in bug 851693, and need to finish it (with some UI to prompt the user).

On the other hand, building them isn't something I'd expect addon authors to know about, but its not that difficult. i.e. see the source here:

http://androidxref.com/4.2.2_r1/xref/frameworks/base/core/java/android/content/Intent.java#6515
Assignee: nobody → lucasr.at.mozilla
Blocks: 972351
Comment on attachment 8375497 [details] [diff] [review]
Introduce ItemType into HomeConfig/Home.jsm (r=margaret)

Use BROWSER item handler by default.
Attachment #8375497 - Flags: review?(margaret.leibovic)
Comment on attachment 8375498 [details] [diff] [review]
Use ViewConfig's item handler to define url opener flags (r=margaret)

There's an unfortunate code duplication here. We should be doing the item handler checks in PanelLayout instead. But doing this here will cause conflicts with jdover's patches in bug 929982, something I'd like to avoid at this point. 

I filed 972351 to fix the code duplication once the folder stuff lands.
Attachment #8375498 - Flags: review?(margaret.leibovic)
Err, I meant jdover's patch in bug 942295.
Comment on attachment 8375497 [details] [diff] [review]
Introduce ItemType into HomeConfig/Home.jsm (r=margaret)

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

Nice. Just make sure to update the commit message - I got confused because there is a separate bug about ItemType :) And we can also add this to the wiki docs, I need to update my add-ons to dogfood these new options we keep adding!
Attachment #8375497 - Flags: review?(margaret.leibovic) → review+
Comment on attachment 8375498 [details] [diff] [review]
Use ViewConfig's item handler to define url opener flags (r=margaret)

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

This duplication makes me think of our comments in bug 968179. Glad you filed a follow-up.
Attachment #8375498 - Flags: review?(margaret.leibovic) → review+
You need to log in before you can comment on or make changes to this bug.