Merge branch 'release/3.8.1'

Author: koke

README.md

@@ -2,33 +2,26 @@ WordPress for iOS
 
 ## Resources
 
-### Developer blog
+### Developer blog & Handbook
 
-http://dev.ios.wordpress.org/
+Blog: http://make.wordpress.org/mobile
+
+Handbook: http://make.wordpress.org/mobile/handbook/
 
 ### Style guide
 
 https://github.com/wordpress-mobile/WordPress-iOS/wiki/WordPress-for-iOS-Style-Guide
 
 ### To report an issue
 
-http://ios.trac.wordpress.org/newticket
-
-You'll need a WordPress.org account. If you don't have one you can
-register here:
-
-http://wordpress.org/support/register.php
+https://github.com/wordpress-mobile/WordPress-iOS/issues
 
 ### Source Code
 
-SVN: http://ios.svn.wordpress.org/
-
-SVN browser: http://ios.trac.wordpress.org/browser
-
-Github mirror: https://github.com/wordpress-mobile/WordPress-iOS/
+GitHub: https://github.com/wordpress-mobile/WordPress-iOS/
 
 ## Building
 
 Starting with changeset 3633 version 3.2, WordPress for iOS uses Cocoapods (http://cocoapods.org/) to manage third party libraries.  Trying to build the project by itself (WordPress.xcproj) after launching will result in an error, as the resources managed by cocoapods are not included.  Instead, launch the workspace by either double clicking on WordPress.xcworkspace file, or launch Xcode and choose File > Open and browse to WordPress.xcworkspace. 
 
-
+You will not be able to connect to WordPress.com hosted blogs when compiling WordPress for iOS yourself.  See the following handbook page for more information: http://make.wordpress.org/mobile/handbook/general-guides/connecting-to-wordpress-com/

WordPress/Classes/EditPostViewController.m

@@ -450,7 +450,9 @@ - (IBAction)showMediaOptions:(id)sender {
     [WPMobileStats flagProperty:StatsPropertyPostDetailClickedMediaOptions forEvent:[self formattedStatEventString:StatsEventPostDetailClosedEditor]];
     PostMediaViewController *vc = [[PostMediaViewController alloc] initWithPost:self.apost];
     vc.postDetailViewController = self;
-    self.navigationItem.title = NSLocalizedString(@"Back", nil);
+    UIBarButtonItem *backButton = [[UIBarButtonItem alloc] initWithTitle:NSLocalizedString(@"Back", @"") style:UIBarButtonItemStyleBordered target:nil action:nil];
+    self.navigationItem.backBarButtonItem = backButton;
+    
     [self.navigationController pushViewController:vc animated:YES];
 }
 
@@ -646,6 +648,7 @@ - (void)populateSelectionsControllerWithCategories {
 			[[CPopoverManager instance] setCurrentPopoverController:popover];
 
         } else {
+            self.navigationItem.backBarButtonItem = [[UIBarButtonItem alloc] initWithTitle:NSLocalizedString(@"Back", @"") style:UIBarButtonItemStyleBordered target:nil action:nil];
 			self.editMode = EditPostViewControllerModeEditPost;
 			[self.navigationController pushViewController:segmentedTableViewController animated:YES];
 		}

WordPress/Classes/GeneralWalkthroughViewController.m

@@ -549,6 +549,7 @@ - (void)addPage1Controls
     if (_pageControl == nil) {
         // The page control adds a bunch of extra space for padding that messes with our calculations.
         _pageControl = [[UIPageControl alloc] init];
+        [_pageControl addTarget:self action:@selector(pageNumberChanged:) forControlEvents:UIControlEventValueChanged];
         _pageControl.numberOfPages = 3;
         [_pageControl sizeToFit];
         [WPNUXUtility configurePageControlTintColors:_pageControl];
@@ -950,6 +951,15 @@ - (void)flagPageViewed:(NSUInteger)pageViewed
     }
 }
 
+- (void)pageNumberChanged:(UIPageControl *)pageControl
+{
+    NSInteger pageNumber = pageControl.currentPage;
+    
+    [_scrollView setContentOffset:CGPointMake(_viewWidth * pageNumber, 0) animated:YES];
+    
+    _currentPage = pageNumber;
+}
+
 - (void)showCompletionWalkthrough
 {
     LoginCompletedWalkthroughViewController *loginCompletedViewController = [[LoginCompletedWalkthroughViewController alloc] init];

WordPress/Classes/NewNotificationsTableViewCell.m

@@ -130,8 +130,9 @@ + (CGFloat)rowHeightForNotification:(Note *)note andMaxWidth:(CGFloat)maxWidth
     CGRect gravatarImageFrame =  [[self class] gravatarImageFrame];
     CGRect subjectFrame = [[self class] subjectFrameForNotification:note leftFrame:gravatarImageFrame andMaxWidth:maxWidth];
     CGRect detailFrame = [[self class] detailFrameForNotification:note leftFrame:gravatarImageFrame topFrame:subjectFrame andMaxWidth:maxWidth];
-    
-    return CGRectGetMaxY(detailFrame) + NotificationCellStandardOffset;
+
+    CGFloat bottom = MAX(CGRectGetMaxY(gravatarImageFrame), CGRectGetMaxY(detailFrame));
+    return bottom + NotificationCellStandardOffset;
 }
 
 - (void)prepareForReuse {

WordPress/Classes/Note.m

@@ -204,8 +204,6 @@ - (void)syncAttributes:(NSDictionary *)noteData {
 }
 
 - (void)updateAttributes:(NSDictionary *)noteData {
-    self.payload = [noteData JSONData];
-    self.noteData = [self.payload mutableObjectFromJSONData];
     if ([noteData objectForKey:@"type"]) {
         self.type = [noteData objectForKey:@"type"];
     }

WordPress/Classes/PanelNavigationController.m

@@ -34,6 +34,7 @@ @interface PanelNavigationController () <UIGestureRecognizerDelegate>
 @property (nonatomic, strong) NSMutableArray *detailViewWidths;
 @property (nonatomic, strong) UIButton *detailTapper;
 @property (nonatomic, strong) UIPanGestureRecognizer *panner;
+@property (nonatomic, strong) UIScreenEdgePanGestureRecognizer *edgePanner;
 @property (nonatomic, strong) UIView *popPanelsView, *menuView;
 @property (nonatomic, strong) UIImageView *sidebarBorderView;
 @property (nonatomic, strong) UIButton *notificationButton, *menuButton;
@@ -112,17 +113,6 @@ @implementation PanelNavigationController {
     BOOL _pushing;
 }
 
-@synthesize detailViewController = _detailViewController;
-@synthesize masterViewController = _masterViewController;
-@synthesize navigationController = _navigationController;
-@synthesize detailViewContainer = _detailViewContainer;
-@synthesize detailViewControllers = _detailViewControllers;
-@synthesize detailViews = _detailViews;
-@synthesize detailViewWidths = _detailViewWidths;
-@synthesize detailTapper = _detailTapper;
-@synthesize panner = _panner;
-@synthesize popPanelsView = _popPanelsView;
-@synthesize sidebarBorderView = _sidebarBorderView;
 @synthesize delegate;
 
 CGFloat const PanelNavigationControllerStatusBarViewHeight = 20.0;
@@ -957,14 +947,14 @@ - (void)closeSidebarAnimated:(BOOL)animated {
         // posts editor, we won't have a blue bar sitting at the top.
         self.statusBarBackgroundView.backgroundColor = [UIColor clearColor];
     }];
-
 
     if(IS_IPHONE && !self.presentedViewController) {
         [SoundUtil playSwipeSound];
     }
 }
 
 - (void)closeSidebarWithVelocity:(CGFloat)velocity {
+    _panned = NO;
     [self enableDetailView];
     [self setStackOffset:(DETAIL_LEDGE_OFFSET - DETAIL_OFFSET) withVelocity:velocity];
 
@@ -1038,6 +1028,7 @@ - (void)enableDetailView {
     }
 
     self.detailTapper = nil;
+    _panned = NO;
 
     // Restore scroll to top behavior to detail view
     [self setScrollsToTop:NO forView:self.masterView];
@@ -1091,13 +1082,27 @@ - (BOOL)gestureRecognizerShouldBegin:(UIGestureRecognizer *)gestureRecognizer {
 }
 
 - (BOOL)gestureRecognizer:(UIGestureRecognizer *)gestureRecognizer shouldReceiveTouch:(UITouch *)touch {
+    // ARD - I'm not sure this is the right spot for the panned = NO reset
     if (gestureRecognizer == self.panner || gestureRecognizer.view == self.detailTapper) {
         _panOrigin = _stackOffset;
         _panned = NO;
     }
+    
+    if (gestureRecognizer == self.edgePanner && [touch.view isDescendantOfView:self.detailViewController.view] == NO) {
+        return NO;
+    }
+    
     return YES;
 }
 
+- (void)edgePanned:(UIScreenEdgePanGestureRecognizer *)sender
+{
+    if (sender.state == UIGestureRecognizerStateEnded) {
+        [self showSidebar];
+        _panned = YES;
+    }
+}
+
 - (void)panned:(UIPanGestureRecognizer *)sender {
     /*
      _stackOffset is how many pixels the views are dragged to the left from the initial (sidebar open) position
@@ -1238,13 +1243,28 @@ - (void)addPanner {
     } else {
         [self.view addGestureRecognizer:panner];
     }
+    
+    if (IS_IOS7) {
+        UIScreenEdgePanGestureRecognizer *edgePanner = [[UIScreenEdgePanGestureRecognizer alloc] initWithTarget:self action:@selector(edgePanned:)];
+        edgePanner.edges = UIRectEdgeLeft;
+        edgePanner.cancelsTouchesInView = YES;
+        edgePanner.delegate = self;
+        self.edgePanner = edgePanner;
+        [self.view addGestureRecognizer:edgePanner];
+    }
 }
 
 - (void)removePanner {
     if (self.panner) {
         [self.panner.view removeGestureRecognizer:self.panner];
     }
+    
+    if (self.edgePanner) {
+        [self.edgePanner.view removeGestureRecognizer:self.panner];
+    }
+    
     self.panner = nil;
+    self.edgePanner = nil;
 }
 
 - (void)setFrameForViewController:(UIViewController *)viewController {
@@ -1598,6 +1618,8 @@ - (void)pushViewController:(UIViewController *)viewController animated:(BOOL)ani
     // Set the panelNavigation before any of the view methods are relayed for iOS4.
     [viewController setPanelNavigationController:self];
 
+    _panned = NO;
+    
     if (self.navigationController) {
         [self.navigationController pushViewController:viewController animated:animated];
     } else {

WordPress/Classes/PostSettingsViewController.m

@@ -892,7 +892,7 @@ - (void)tableView:(UITableView *)aTableView didSelectRowAtIndexPath:(NSIndexPath
                     case 1:
                         [WPMobileStats trackEventForWPCom:[self formattedStatEventString:StatsEventPostDetailSettingsClickedRemoveFeaturedImage]];
                         actionSheet = [[UIActionSheet alloc] initWithTitle:NSLocalizedString(@"Remove this Featured Image?", @"Prompt when removing a featured image from a post") delegate:self cancelButtonTitle:NSLocalizedString(@"Cancel", "Cancel a prompt") destructiveButtonTitle:NSLocalizedString(@"Remove", @"Remove an image/posts/etc") otherButtonTitles:nil];
-                        [actionSheet showFromRect:cell.frame inView:self.view animated:YES];
+                        [actionSheet showFromRect:cell.frame inView:tableView animated:YES];
                         break;
                 }
             } else {

WordPress/Classes/ReaderPostDetailViewController.m

@@ -175,6 +175,8 @@ - (void)viewWillAppear:(BOOL)animated {
 	if (IS_IPAD)
         [self.panelNavigationController setToolbarHidden:NO forViewController:self animated:NO];
 
+    [self.post addObserver:self forKeyPath:@"isReblogged" options:NSKeyValueObservingOptionNew context:@"reblogging"];
+
 	[_headerView updateLayout];
 	[self showStoredComment];
 }
@@ -190,8 +192,6 @@ - (void)viewDidAppear:(BOOL)animated {
     if (lastSynced == nil || ABS([lastSynced timeIntervalSinceNow]) > ReaderPostDetailViewControllerRefreshTimeout) {
 		[self syncWithUserInteraction:NO];
     }
-	
-	[self.post addObserver:self forKeyPath:@"isReblogged" options:NSKeyValueObservingOptionNew context:@"reblogging"];
 }
 
 

WordPress/Classes/ReaderPostsViewController.m

@@ -257,6 +257,12 @@ - (void)handleTopicsButtonTapped:(id)sender {
 	ReaderTopicsViewController *controller = [[ReaderTopicsViewController alloc] initWithStyle:UITableViewStyleGrouped];
 	controller.delegate = self;
     if (IS_IPAD) {
+        if (_popover) {
+            [_popover dismissPopoverAnimated:YES];
+            _popover = nil;
+            return;
+        }
+        
         _popover = [[UIPopoverController alloc] initWithContentViewController:controller];
         _popover.popoverBackgroundViewClass = [WPPopoverBackgroundView class];
 

WordPress/Classes/WPWebViewController.m

@@ -134,7 +134,7 @@ - (void)viewDidLoad
         toolbar.barTintColor = [WPStyleGuide littleEddieGrey];
         toolbar.tintColor = [UIColor whiteColor];        
     } else {
-        UIColor *color = [UIColor UIColorFromHex:0x464646];
+        UIColor *color = [UIColor whiteColor];
         backButton.tintColor = color;
         forwardButton.tintColor = color;
         refreshButton.tintColor = color;