Opened on 02/10/2016 at 10:52:16 AM

Closed on 08/15/2017 at 01:13:35 PM

#3642 closed change (worksforme)

Unstick Adblock Browser Notification

Reported by: mario Assignee: ashephard
Priority: P3 Milestone:
Module: Adblock-Plus-for-iOS/macOS Keywords: salsita
Cc: Blocked By:
Blocking: Platform: iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by mario)


If you open Adblock Plus for iOS on a device where Adblock Browser for iOS isn't installed, the App will show a notification at the bottom of the settings screen. On smaller displays (iPhone 5) this notification overlays the bottom settings items due to the fact that we added numerous new items for the new release as shown in this screenshot.
These items are now hard to reach which is why we want to unstick this notification and make it part of the scrollable content.

What to change

Make the Adblock Browser notification in the settings screen part of the content.

  • It shouldn't be sticky anymore and shouldn't overlay settings items.
  • Thus is should be displayed as the last item in the settings screen (below the "More" item) and move with the rest of the content while scrolling.

Remark for testers

This can only be tested on an iPhone 5. iPhone 6 and above have bigger screens and thus more space for content. The overlapping only happens on smaller screens as shown in this screenshot.
Also you need to make sure that Adblock Browser for iOS isn't installed for the notification to be shown.

Attachments (3)

abp_abb_notice.png (83.8 KB) - added by mario on 02/17/2016 at 04:07:29 PM.
Simulator Screen Shot 14 Aug 2017, 15.52.43.png (104.4 KB) - added by ashephard on 08/14/2017 at 01:55:41 PM.
Screenshot of iPhone SE demonstrating fully visible About button.
Simulator Screen Shot 15 Aug 2017, 14.52.03.png (95.2 KB) - added by ashephard on 08/15/2017 at 12:54:06 PM.
Increased number of table view options. Banner is not blocking any of possible table view options.

Download all attachments as: .zip

Change History (12)

comment:1 Changed on 02/10/2016 at 11:09:02 AM by mario

  • Component changed from Adblock-Browser-for-iOS to Adblock-Plus-for-iOS
  • Platform changed from Adblock Browser for iOS to iOS

Changed on 02/17/2016 at 04:07:29 PM by mario

comment:2 Changed on 02/17/2016 at 04:09:14 PM by mario

  • Description modified (diff)

comment:3 Changed on 02/17/2016 at 04:09:21 PM by mario

  • Ready set

comment:4 Changed on 03/16/2016 at 11:00:36 AM by mario

  • Priority changed from Unknown to P3

comment:5 Changed on 08/14/2017 at 01:53:02 PM by ashephard

  • Owner set to ashephard

comment:6 Changed on 08/14/2017 at 01:55:06 PM by ashephard

The content isn't hidden behind the Adblock Browser notification label. The user has the ability to scroll the view and the 'About' option is fully visible. Would advise for the ticket to be closed. See attached image. Tested on iPhone SE (Same aspect/screen size as iPhone 5).

Changed on 08/14/2017 at 01:55:41 PM by ashephard

Screenshot of iPhone SE demonstrating fully visible About button.

comment:7 Changed on 08/14/2017 at 03:11:44 PM by mario

@ashephard, with #3241 and later on #4661 we'll introduce additional settings items, which will eventually push the bottom most item under the notification about ABB. Installing the current devbuild (containing #3241) on an iPhone SE should already result in this.

comment:8 Changed on 08/15/2017 at 12:53:30 PM by ashephard

@mario I've checked the latest dev build and also added more options to the table view for testing this. The notification banner doesn't overlap with any of the table view settings (see attachment). Nothing needs to be edited code-wise, would recommend closing this ticket.

Changed on 08/15/2017 at 12:54:06 PM by ashephard

Increased number of table view options. Banner is not blocking any of possible table view options.

comment:9 Changed on 08/15/2017 at 01:13:35 PM by mario

  • Resolution set to worksforme
  • Status changed from new to closed

It appears like this was unintentionally fixed while introducing other changes to the settings overview. Rejecting this change for now, but keeping it on the radar, just in case.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from ashephard.
Note: See TracTickets for help on using tickets.