[MediaWiki-commits] [Gerrit] Table of contents refinements. - change (apps...wikipedia)

2014-07-07 Thread Mhurd (Code Review)
Mhurd has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/144597

Change subject: Table of contents refinements.
..

Table of contents refinements.

Change-Id: I5b2b728fb804034919460e7a5afd72fe3eb9ca56
---
M wikipedia/Base.lproj/Main_iPhone.storyboard
M wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
M wikipedia/View Controllers/TableOfContents/TOCViewController.m
M wikipedia/View Controllers/WebView/WebViewController.m
M wikipedia/en.lproj/Localizable.strings
M wikipedia/qqq.lproj/Localizable.strings
6 files changed, 117 insertions(+), 65 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/apps/ios/wikipedia 
refs/changes/97/144597/1

diff --git a/wikipedia/Base.lproj/Main_iPhone.storyboard 
b/wikipedia/Base.lproj/Main_iPhone.storyboard
index cfd1212..7de9052 100644
--- a/wikipedia/Base.lproj/Main_iPhone.storyboard
+++ b/wikipedia/Base.lproj/Main_iPhone.storyboard
@@ -1,5 +1,5 @@
 ?xml version=1.0 encoding=UTF-8 standalone=no?
-document type=com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB 
version=3.0 toolsVersion=5056 systemVersion=13D65 
targetRuntime=iOS.CocoaTouch propertyAccessControl=none useAutolayout=YES 
initialViewController=1qV-3k-dN1
+document type=com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB 
version=3.0 toolsVersion=5056 systemVersion=13E28 
targetRuntime=iOS.CocoaTouch propertyAccessControl=none useAutolayout=YES 
initialViewController=1qV-3k-dN1
 dependencies
 deployment defaultVersion=1536 identifier=iOS/
 plugIn identifier=com.apple.InterfaceBuilder.IBCocoaTouchPlugin 
version=3733/
@@ -108,7 +108,7 @@
 /constraints
 /view
 /subviews
-color key=backgroundColor 
red=0.22052375637755106 green=0.22052375637755106 
blue=0.22052375637755106 alpha=1 colorSpace=calibratedRGB/
+color key=backgroundColor 
red=0.047058828175067902 green=0.047058828175067902 
blue=0.047058828175067902 alpha=1 colorSpace=deviceRGB/
 constraints
 constraint firstItem=VGj-z3-9mh 
firstAttribute=leading secondItem=sgu-W1-m18 secondAttribute=leading 
id=jVV-fN-1Va/
 constraint firstAttribute=bottom 
secondItem=VGj-z3-9mh secondAttribute=bottom id=jvc-8X-PKF/
@@ -1254,9 +1254,9 @@
 constraints
 constraint firstItem=WeL-Mj-Zsh 
firstAttribute=top secondItem=C8y-0k-FBq secondAttribute=bottom 
id=3Gh-gP-O56/
 constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=kh9-bI-dsS secondAttribute=leading 
id=3Zs-jL-a48/
-constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=F4Q-Xc-ImV secondAttribute=leading 
id=87k-zE-vFH/
-constraint firstAttribute=trailing 
secondItem=F4Q-Xc-ImV secondAttribute=trailing id=D19-HH-fX5/
+constraint firstItem=WeL-Mj-Zsh 
firstAttribute=trailing secondItem=F4Q-Xc-ImV secondAttribute=trailing 
id=J2p-jy-zrW/
 constraint firstAttribute=trailing 
secondItem=WeL-Mj-Zsh secondAttribute=trailing id=QSr-uN-iN4/
+constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=F4Q-Xc-ImV secondAttribute=leading 
id=akc-jF-1Pw/
 constraint firstAttribute=bottom 
secondItem=F4Q-Xc-ImV secondAttribute=bottom placeholder=YES 
id=bOy-uv-taw/
 constraint firstAttribute=bottom 
secondItem=WeL-Mj-Zsh secondAttribute=bottom id=myK-xK-1sX/
 /constraints
diff --git a/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m 
b/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
index 8e9c5d7..b5b559c 100644
--- a/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
+++ b/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
@@ -9,6 +9,9 @@
 #import WMF_Colors.h
 #import UIView+RemoveConstraints.h
 #import Section+LeadSection.h
+#import NSString+FormattedAttributedString.h
+#import WikipediaAppUtils.h
+#import SessionSingleton.h
 
 @interface TOCSectionCellView(){
 
@@ -16,12 +19,8 @@
 
 @property (nonatomic, strong) UILabel *titleLabel;
 
-@property (nonatomic) CGFloat indentMargin;
-@property (nonatomic) CGFloat indentMarginMin;
-
-@property (nonatomic) CGFloat imageIndentMargin;
-@property (nonatomic) CGFloat imageIndentMarginMin;
-@property (nonatomic) CGFloat imageMargin;
+@property (nonatomic) UIEdgeInsets cellMargin;
+@property (nonatomic) UIEdgeInsets imageMargin;
 @property (nonatomic) CGSize imageSize;
 
 @property (nonatomic, strong) NSNumber *tocLevel;
@@ -49,14 +48,9 @@
 
 [self addSubview:self.titleLabel];
 

[MediaWiki-commits] [Gerrit] Table of contents refinements. - change (apps...wikipedia)

2014-07-07 Thread Brion VIBBER (Code Review)
Brion VIBBER has submitted this change and it was merged.

Change subject: Table of contents refinements.
..


Table of contents refinements.

Change-Id: I5b2b728fb804034919460e7a5afd72fe3eb9ca56
---
M wikipedia/Base.lproj/Main_iPhone.storyboard
M wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
M wikipedia/View Controllers/TableOfContents/TOCViewController.m
M wikipedia/View Controllers/WebView/WebViewController.m
M wikipedia/en.lproj/Localizable.strings
M wikipedia/qqq.lproj/Localizable.strings
6 files changed, 117 insertions(+), 65 deletions(-)

Approvals:
  Brion VIBBER: Verified; Looks good to me, approved



diff --git a/wikipedia/Base.lproj/Main_iPhone.storyboard 
b/wikipedia/Base.lproj/Main_iPhone.storyboard
index cfd1212..7de9052 100644
--- a/wikipedia/Base.lproj/Main_iPhone.storyboard
+++ b/wikipedia/Base.lproj/Main_iPhone.storyboard
@@ -1,5 +1,5 @@
 ?xml version=1.0 encoding=UTF-8 standalone=no?
-document type=com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB 
version=3.0 toolsVersion=5056 systemVersion=13D65 
targetRuntime=iOS.CocoaTouch propertyAccessControl=none useAutolayout=YES 
initialViewController=1qV-3k-dN1
+document type=com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB 
version=3.0 toolsVersion=5056 systemVersion=13E28 
targetRuntime=iOS.CocoaTouch propertyAccessControl=none useAutolayout=YES 
initialViewController=1qV-3k-dN1
 dependencies
 deployment defaultVersion=1536 identifier=iOS/
 plugIn identifier=com.apple.InterfaceBuilder.IBCocoaTouchPlugin 
version=3733/
@@ -108,7 +108,7 @@
 /constraints
 /view
 /subviews
-color key=backgroundColor 
red=0.22052375637755106 green=0.22052375637755106 
blue=0.22052375637755106 alpha=1 colorSpace=calibratedRGB/
+color key=backgroundColor 
red=0.047058828175067902 green=0.047058828175067902 
blue=0.047058828175067902 alpha=1 colorSpace=deviceRGB/
 constraints
 constraint firstItem=VGj-z3-9mh 
firstAttribute=leading secondItem=sgu-W1-m18 secondAttribute=leading 
id=jVV-fN-1Va/
 constraint firstAttribute=bottom 
secondItem=VGj-z3-9mh secondAttribute=bottom id=jvc-8X-PKF/
@@ -1254,9 +1254,9 @@
 constraints
 constraint firstItem=WeL-Mj-Zsh 
firstAttribute=top secondItem=C8y-0k-FBq secondAttribute=bottom 
id=3Gh-gP-O56/
 constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=kh9-bI-dsS secondAttribute=leading 
id=3Zs-jL-a48/
-constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=F4Q-Xc-ImV secondAttribute=leading 
id=87k-zE-vFH/
-constraint firstAttribute=trailing 
secondItem=F4Q-Xc-ImV secondAttribute=trailing id=D19-HH-fX5/
+constraint firstItem=WeL-Mj-Zsh 
firstAttribute=trailing secondItem=F4Q-Xc-ImV secondAttribute=trailing 
id=J2p-jy-zrW/
 constraint firstAttribute=trailing 
secondItem=WeL-Mj-Zsh secondAttribute=trailing id=QSr-uN-iN4/
+constraint firstItem=WeL-Mj-Zsh 
firstAttribute=leading secondItem=F4Q-Xc-ImV secondAttribute=leading 
id=akc-jF-1Pw/
 constraint firstAttribute=bottom 
secondItem=F4Q-Xc-ImV secondAttribute=bottom placeholder=YES 
id=bOy-uv-taw/
 constraint firstAttribute=bottom 
secondItem=WeL-Mj-Zsh secondAttribute=bottom id=myK-xK-1sX/
 /constraints
diff --git a/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m 
b/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
index 8e9c5d7..b5b559c 100644
--- a/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
+++ b/wikipedia/View Controllers/TableOfContents/TOCSectionCellView.m
@@ -9,6 +9,9 @@
 #import WMF_Colors.h
 #import UIView+RemoveConstraints.h
 #import Section+LeadSection.h
+#import NSString+FormattedAttributedString.h
+#import WikipediaAppUtils.h
+#import SessionSingleton.h
 
 @interface TOCSectionCellView(){
 
@@ -16,12 +19,8 @@
 
 @property (nonatomic, strong) UILabel *titleLabel;
 
-@property (nonatomic) CGFloat indentMargin;
-@property (nonatomic) CGFloat indentMarginMin;
-
-@property (nonatomic) CGFloat imageIndentMargin;
-@property (nonatomic) CGFloat imageIndentMarginMin;
-@property (nonatomic) CGFloat imageMargin;
+@property (nonatomic) UIEdgeInsets cellMargin;
+@property (nonatomic) UIEdgeInsets imageMargin;
 @property (nonatomic) CGSize imageSize;
 
 @property (nonatomic, strong) NSNumber *tocLevel;
@@ -49,14 +48,9 @@
 
 [self addSubview:self.titleLabel];
 
-self.indentMargin = 0.0f;
-