Opened on 11/02/2017 at 01:29:08 PM
Closed on 11/21/2017 at 12:48:22 PM
Last modified on 11/29/2017 at 01:28:01 PM
#5977 closed defect (fixed)
[webextension] Adblock Plus menu item opens bubble UI instead of mobile options page
Reported by: | Ross | Assignee: | mjethani |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-3.0.2-for-Firefox |
Module: | Platform | Keywords: | |
Cc: | trev, sebastian, kzar, greiner | Blocked By: | #6037 |
Blocking: | Platform: | Firefox | |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
Description (last modified by kzar)
Environment
ABP 2.99.0.1903beta
Firefox Beta (57) / Android
How to reproduce
- Have ABP webextension build installed on Firefox Beta for Android.
- Navigate to a website (any).
- Select [Firefox menu] > [Adblock Plus] (bottom entry).
Observed behaviour
It opens the bubble UI meant for desktop versions.
Expected behaviour
It should open the mobile options page instead.
What to change
Modify metadata.gecko so the popup is not set in the manifest file. In lib/options.js set the popup programmatically if the platform is not Firefox for Android.
Update buildtools dependency to 1c798cc8b402 so the build script doesn't set one of the popup icons as the popup. (No other relevant changes will be included.)
Hints for testers
- Test the popup opens correctly still for desktop browsers.
- Test that the mobile options page opens for Firefox on Android, instead of the popup.
Attachments (0)
Change History (16)
comment:1 Changed on 11/03/2017 at 02:01:27 PM by trev
comment:3 Changed on 11/05/2017 at 11:46:41 AM by mjethani
- Component changed from User-Interface to Platform
- Description modified (diff)
- Summary changed from [webextension] Issue reporter is not focused on mobile to [webextension] Adblock Plus menu item opens bubble UI instead of mobile options page
comment:4 Changed on 11/05/2017 at 11:47:20 AM by mjethani
- Owner set to mjethani
comment:5 Changed on 11/05/2017 at 11:50:30 AM by mjethani
I should note here that there's a bug in Firefox for Android that prevents us from implementing an ideal fix for this issue.
comment:6 Changed on 11/05/2017 at 11:51:06 AM by mjethani
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:7 Changed on 11/05/2017 at 11:52:53 AM by mjethani
- Cc kzar added
comment:8 Changed on 11/14/2017 at 03:29:49 PM by mjethani
- Blocked By 6037 added
comment:9 Changed on 11/14/2017 at 04:02:19 PM by kzar
- Priority changed from Unknown to P2
- Ready set
comment:10 Changed on 11/16/2017 at 11:12:14 AM by greiner
- Cc greiner added
comment:11 Changed on 11/20/2017 at 04:08:04 PM by mjethani
- Description modified (diff)
comment:12 Changed on 11/21/2017 at 07:35:39 AM by mjethani
- Description modified (diff)
comment:13 Changed on 11/21/2017 at 12:35:01 PM by kzar
- Description modified (diff)
comment:14 Changed on 11/21/2017 at 12:46:22 PM by abpbot
A commit referencing this issue has landed:
Issue 5977 - Set popup programmatically on Firefox
comment:15 Changed on 11/21/2017 at 12:48:22 PM by mjethani
- Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next
- Resolution set to fixed
- Status changed from reviewing to closed
comment:16 Changed on 11/29/2017 at 01:28:01 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Fixed. Mobile options page is now opened on mobile and desktop popup UI still displays as expected.
ABP 3.0.1.1939
Firefox 51 / 57 / Windows 7
Firefox 57 / Galaxy J5 / Android 6.0.1
Chrome 49 / 62 / Windows 7
Opera 36 / 48 / Windows 7
Actually, the issue here is that the bubble is opened in the first place - it isn't meant for mobile. This is supposed to open options.