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)
Background
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)
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: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
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
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.
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).