Bug 1355121 - Add the correct insets for the onboarding view in AS. (#…

…2620) r=sleroux
mozilla-mobile · Apr 12, 2017 · a93cb54 · a93cb54
1 parent 636a921
commit a93cb54
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Client/Frontend/Home/ActivityStreamPanel.swift b/Client/Frontend/Home/ActivityStreamPanel.swift
@@ -154,7 +154,7 @@ extension ActivityStreamPanel {
             case .topSites:
                 return UIDevice.current.userInterfaceIdiom == .pad ? ASPanelUX.SectionInsetsForIpad : 0
             case .highlightIntro:
-                return 0
+                return UIDevice.current.userInterfaceIdiom == .pad ? ASPanelUX.SectionInsetsForIpad : 0
             }
         }
 
@@ -176,7 +176,7 @@ extension ActivityStreamPanel {
             case .topSites:
                 return CGSize(width: frameWidth - inset, height: height)
             case .highlightIntro:
-                return CGSize(width: frameWidth - inset, height: height)
+                return CGSize(width: frameWidth - inset - (ASHorizontalScrollCellUX.MinimumInsets * 2), height: height)
             }
         }
 

diff --git a/Client/Frontend/Widgets/ActivityStreamHighlightCell.swift b/Client/Frontend/Widgets/ActivityStreamHighlightCell.swift
@@ -212,7 +212,7 @@ class HighlightIntroCell: UICollectionViewCell {
         titleLabel.text = Strings.HighlightIntroTitle
         descriptionLabel.text = Strings.HighlightIntroDescription
 
-        let titleInsets = UIEdgeInsets(top: HighlightIntroCellUX.margin, left: HighlightIntroCellUX.margin, bottom: 0, right: 0)
+        let titleInsets = UIEdgeInsets(top: HighlightIntroCellUX.margin, left: 0, bottom: 0, right: 0)
         titleLabel.snp.makeConstraints { make in
             make.leading.top.equalTo(self.contentView).inset(titleInsets)
             make.trailing.equalTo(mainImageView.snp.leading)