Opened on 03/10/2018 at 01:58:30 AM

Closed on 03/16/2018 at 11:34:45 PM

#6462 closed change (fixed)

Clear warnings for storyboard "You're in control" dialog and numeral labels

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

https://codereview.adblockplus.org/29718595/

Description (last modified by dzhang)

Background

Three warnings are reported from the main storyboard that possibly arose from an update in Xcode.

  1. Auto Layout Localization: Trailing constraint is missing, which may cause overlapping with other views.
  1. Ambiguous Layout Group
  • 2 views are vertically ambiguous.
    • Height and vertical position are ambiguous for "Annoying ads are always blocked, while nonintrusive ads are displayed by default.
    • Height is ambiguous for "You're in control".
  1. Ambiguous Layout: Height and vertical position are ambiguous for "Acceptable Ads → Allow some nonintrusive ads".

What to change

Clear warnings using the following actions

  • Add trailing constraints for (2)
  • Make a container view for (3) and copy all controls into the container.

Attachments (0)

Change History (3)

comment:1 Changed on 03/10/2018 at 02:12:11 AM by dzhang

  • Description modified (diff)

comment:2 Changed on 03/10/2018 at 02:12:32 AM by dzhang

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 03/16/2018 at 11:34:45 PM by dzhang

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

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