Shake animation rotating display as well as animating

Question

I have a method to perform a small shake animation. This animation works somewhat, but it rotates every time I call it from landscape position. When the animation is called while the simulator is in landscape, the entire view rotates itself to portrait, and then performs the animation. The rotation itself is not animated, it just changes suddenly and with no delay. Everything in the view shifts as well, all buttons, text fields, image views, etc.

Animation Code:

- (void)shakeView
{   
    CGFloat t = 8.0;
    CGAffineTransform translateRight = CGAffineTransformTranslate(CGAffineTransformIdentity, 0.0, t);
    CGAffineTransform translateLeft = CGAffineTransformTranslate(CGAffineTransformIdentity, 0.0, -t);

    self.view.transform = translateLeft;

    [UIView animateWithDuration:0.07 delay:0.0 options:UIViewAnimationOptionAutoreverse|UIViewAnimationOptionRepeat animations:^{[UIView setAnimationRepeatCount:3.0];

        self.view.transform = translateRight;
    }
        completion:^(BOOL finished){
        if (finished)
            {
                [UIView animateWithDuration:0.05 delay:0.0 options:UIViewAnimationOptionBeginFromCurrentState animations:^{self.view.transform = CGAffineTransformIdentity;
            }
        completion:NULL];
        }
    }];
}

Honestly, I don't do much with animations, so I have no idea what I am doing wrong here, much less what else I should try or where I should look for an answer.

I am looking to keep the animation while also keeping the orientation.

I don't see any problem..its completely working fine... there might be problem in some other part of your code. — Prashant Nikam, Oct 22 '13 at 06:29
@Xman, did you run the animation in landscape? My entire UIView, all buttons, text fields, and image views flip to portrait while the simulator stays in landscape. — CaptJak, Oct 22 '13 at 16:02
So I ended up using the animation given here: http://stackoverflow.com/a/3858202/2535467. But the above code has not been fixed. — CaptJak, Oct 25 '13 at 14:25

score 1 · Accepted Answer · edited May 23 '17 at 12:14

The issue is that the view has a non identity transform on it already. In order for a view to display properly in landscape, the system applies a transform on it that rotates the view. The transform of the view is CGAffineTransformIdentity ONLY when the device is in UIDeviceOrientationPortrait. This orientation rotation ONLY applies to the root child of the application, the first subview of the window. When you are initializing your translation animation with:

CGAffineTransform translateRight = CGAffineTransformTranslate(CGAffineTransformIdentity, 0.0, t);

You are telling the rotated view to ignore its rotation and snap back to its orientation in portrait mode. You can solve this issue by doing one of two things:

Make the animated view a subview of the root view. This has the advantage that the system will still handle rotations for free, and there isn't any chance that your animation will screw up the the orientation changes. Furthermore, the code will be simpler and easier to maintain in the end.
Store the initial transformation of the view, and apply all transformations to it.

I'll try to do #2 here.

- (void)shakeView
{   
    CGFloat t = 8.0;

    //store the initial transform and use it to create the animated transforms
    CGAffineTransform initialTransform = self.view.transform;
    CGAffineTransform translateRight = CGAffineTransformTranslate(initialTransform, 0.0, t);
    CGAffineTransform translateLeft = CGAffineTransformTranslate(initialTransform, 0.0, -t);

    self.view.transform = translateLeft;

    [UIView animateWithDuration:0.07 
                          delay:0.0 
                        options:UIViewAnimationOptionAutoreverse|UIViewAnimationOptionRepeat 
                     animations:^{
        [UIView setAnimationRepeatCount:3.0];

        self.view.transform = translateRight;
                                }
                     completion:^(BOOL finished){
        if (finished)
        {
            [UIView animateWithDuration:0.05 
                                  delay:0.0 
                                options:UIViewAnimationOptionBeginFromCurrentState 
                             animations:^{
                //restore the initial transform
                self.view.transform = initialTransform;
                                         }
                             completion:NULL];
        }
    }];
}

Note - As of iOS 8, the root view of the UIWindow is no longer transformed on rotation. Instead the bounds of the UIWindow itself changes. For another question about this, see here: UIWindow frame in ios 8 different from ios 7 in Landscape

Shake animation rotating display as well as animating

1 Answers1