Closed Bug 1019613 Opened 8 years ago Closed 8 years ago

First tap in the Marketplace page will open the same page in a new tab

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(firefox30 affected, firefox31 affected, firefox32 affected, firefox33 verified, fennec30+)

RESOLVED FIXED
Firefox 33
Tracking Status
firefox30 --- affected
firefox31 --- affected
firefox32 --- affected
firefox33 --- verified
fennec 30+ ---

People

(Reporter: csuciu, Assigned: wesj)

References

Details

Attachments

(1 file)

Logging a separate bug for the issue reported in https://bugzilla.mozilla.org/show_bug.cgi?id=985867#c25.  

Steps:
1. Open marketplace.firefox.com via Menu > Tools > Apps
2. Tap anywhere on the page 

Results: marketplace.firefox.com is open in a new tab.

Note: This only happens at first tap on the page
It looks like we need to add the same hack for about:apps that we landed for about:addons.

Are other pages affected, like about:downloads? I guess that doesn't have a link to a different webpage in the same way about:addons and about:apps do.
Flags: needinfo?(wjohnston)
tracking-fennec: --- → ?
We should probably address the core issue. What bug is that?
(In reply to Aaron Train [:aaronmt] from comment #2)
> We should probably address the core issue. What bug is that?

An attempt for a real fix was made in bug 985867, but that got backed out. I think we should think about landing a similar hack for about:apps, then file a follow-up bug for a real fix, since I don't see a bug filed about that right now.
Assignee: nobody → wjohnston
tracking-fennec: ? → 30+
Attached patch Hack patchSplinter Review
Here's the same hack fix we used for addons. I'll file a separate bug to fix this correctly.
Attachment #8435111 - Flags: review?(mark.finkle)
Flags: needinfo?(wjohnston)
Filed bug 1021148 for a real fix.
Depends on: 1021148
Comment on attachment 8435111 [details] [diff] [review]
Hack patch

> #ifndef MOZ_ANDROID_SYNTHAPKS
>   ContextMenus.init();
> #endif
>+
>+
>+  // XXX - Hack to fix bug 985867 for now
>+  document.addEventListener("touchstart", function() { });

One blank line is enough
Attachment #8435111 - Flags: review?(mark.finkle) → review+
Verified fixed on Nightly 33.0a1 (2014-06-18)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.