Проблема с кастомным TabBarController в ios7 [видеовопрос]?

Question

[deleted-mezhevikin]

Использую кастомный таббар — AKTabBarController

В ios 7 наблюдается бага, если перейти в котроллер с скрытым таббаром и вернуться назад, то невозможно проскроллить таблицу до конца.


Высота contentView расчитывается в AKTabBarView.m и расчитывается правильно.


C чем может быть связана подобная бага в ios7?

- (void)layoutSubviews
{
    [super layoutSubviews];
    _tabBar.frame = (CGRect) {
        .origin.x = _tabBar.frame.origin.x,
        .origin.y = (self.tabBarPosition == AKTabBarPositionTop) ? 0.0
                                                                 : CGRectGetHeight(self.bounds) - CGRectGetHeight(_tabBar.bounds),
        .size = _tabBar.frame.size
    };

    _contentView.frame = (CGRect) {
        .origin.x = 0,
        .origin.y = (self.tabBarPosition == AKTabBarPositionTop) ? CGRectGetMaxY(_tabBar.frame) : 0,
        .size.width = CGRectGetWidth(self.bounds),
        .size.height = CGRectGetHeight(self.bounds) - ((!_isTabBarHidding) ? CGRectGetHeight(_tabBar.bounds) : 0)
    };
    NSLog(@"%f", _contentView.frame.size.height);
}

p.s. issue на гитхабе создал 2 недели назад, разработчик так и не ответил

Answer 1

[Ан]

А можно посмотреть исходники этого проекта?
У Вас после перехода на след. экран высота frame'а таблицы увеличивается на высоту tapBar'a. Такое чувство что при обратной анимации эта высота не сбрасывается, и так и остается большей на высоту tapBar'a.
Таблица, это же _contentView?
если так, то посмотрите чему равно
_isTabBarHidding
при выполнении обратной анимации в строке
.size.height = CGRectGetHeight(self.bounds) — ((!_isTabBarHidding)? CGRectGetHeight(_tabBar.bounds): 0)
есть предположение, что _isTabBarHidding == YES (хотя должно быть NO).
Если так, то скорей всего layoutSubview вызывается раньше, чем Вы обновляете _isTabBarHidding, и Вам нужно принудительно послать базовой вьюхе setNeedsLayout, после установки _isTabBarHidding == NO
Но повторюсь, без исходников

Comments

[deleted-mezhevikin]

ссылка на исходник была в вопросе :)
github.com/alikaragoz/AKTabBarController
пример в папке /AKTabBar Example/

[deleted-mezhevikin]

если правильно понял ваш совет в классе AKTabBarController.m есть метод который вызывается при смене контроллера, добавил
[vc.view setNeedsLayout]; после того как таббар опять показан, но не помогло

- (void)navigationController:(UINavigationController *)navigationController willShowViewController:(UIViewController *)viewController animated:(BOOL)animated
{
    if (!prevViewControllers)
        prevViewControllers = [navigationController viewControllers];
    
    
    // We detect is the view as been push or popped
    BOOL pushed;
    
    if ([prevViewControllers count] <= [[navigationController viewControllers] count])
        pushed = YES;
    else
        pushed = NO;
    
    // Logic to know when to show or hide the tab bar
    BOOL isPreviousHidden, isNextHidden;
    
    isPreviousHidden = [[prevViewControllers lastObject] hidesBottomBarWhenPushed];
    isNextHidden = [viewController hidesBottomBarWhenPushed];
    
    prevViewControllers = [navigationController viewControllers];
    
    UIViewController *vc = [prevViewControllers lastObject];
    NSLog(@"%@", vc);

    if (!isPreviousHidden && !isNextHidden)
        return;
    
    else if (!isPreviousHidden && isNextHidden)
    {
        [self hideTabBar:(pushed ? AKShowHideFromRight : AKShowHideFromLeft) animated:animated];
    }
    else if (isPreviousHidden && !isNextHidden)
    {
        NSLog(@"2");
        [self showTabBar:(pushed ? AKShowHideFromRight : AKShowHideFromLeft) animated:animated];
        [vc.view setNeedsLayout]; // <======= ПО ВАШЕМУ СОВЕТУ
    }
    else if (isPreviousHidden && isNextHidden)
    {
        return;
    }
    
}

[Ан]

Не совсем, на самом деле там такой код уже есть (я отрыл исходники)

 tabBarView.isTabBarHidding = NO;
 [tabBarView setNeedsLayout]; ///<-------вот эта строка

Что касается самой проблемы. Вот посмотрите, ее можно решить, добавив в AKTabBarController.m (в секцию делегата NavigationController'a) метод

-(void) navigationController:(UINavigationController *)navigationController didShowViewController:(UIViewController *)viewController animated:(BOOL)animated{
    CGRect viewControllerFrame = viewController.view.frame;
    viewControllerFrame.size.height = navigationController.view.frame.size.height-viewControllerFrame.origin.y;
    viewController.view.frame = viewControllerFrame;
}

НО, это какое-то дичайшее исправление. Мне кажется есть что-то не только лучше, но и правильнее.
Суть проблемы в чем. При переходе на второй экран по ячейке, NavigationController увеличивает свой frame на размер bottomBar'a (попутно увеличивая frame'ы вьюшех своих дочерних ViewController'ов (в нашем случае это FirstViewController).
А вот при обратной анимации этого не происходит, т.е. frame navigationController.view изменяется без проблемм, а у firstViewController'a, который станет видимым, почему-то нет.
не помогает даже выделенная строчка

_contentView.frame = (CGRect) {
        .origin.x = 0,
        .origin.y = (self.tabBarPosition == AKTabBarPositionTop) ? CGRectGetMaxY(_tabBar.frame) : 0,
        .size.width = CGRectGetWidth(self.bounds),
        .size.height = CGRectGetHeight(self.bounds) - ((!_isTabBarHidding) ? CGRectGetHeight(_tabBar.bounds) : 0)
    };
    [_contentView setNeedsLayout]; ///<--- в данном случае это вьюшка navigationcontroller'a и по идее она должна перересайзить вьюшку firstViewController'а

почему так происходит я не знаю. То ли баг ios 7 (сомневаюсь), то ли не выполнена какая-то рекомендация по переходу с ios 6 на ios 7.
Давайте подождем кого-то, кто уже имел дело с ios7… может он скажет что-то.
К слову, в ios 6 все хорошо, потому как вьюшка firstViewController'а таки ресайзится.
Самому очень хочется разобраться, в чем же причина :)

[deleted-mezhevikin]

спасибо за помощь, работает, хотя действительно костыль :)

[Ан]

Я вот еще чего нарыл. оказывает фрейм firstViewController'a изменяет сам UINavigationController. В iOS 7 добавили возможность задавать пользовательскую анимацию смены контроллера у UINavigationController'a. Для этого в одном из новых методов делегата нужно вернуть пользовательский Animation Transition. По-умолчанию используется некая стандартная анимация, которая и поганит frame вьюхи. Т.е. окончательно все выглядит так - в методе navigationController: willShowViewController: animated: FirsViewController'у вроде как устанавливается ожидаемое значение frame - стандартная Transition выполняет анимацию перемещения между двух контроллеров, но делает детский (child) viewController полноэкранным, что в ios 7 как бы норма. - мой костыль в didShowViewController возвращает все обратно Таким образом, можно еще попытаться сделать такие два финта ушами 1) переопределить стандартную Transition Animation у NavigationController'a и тем самым убрать нежелательное изменение frame 2) посмотреть на bottomLayoutGuide у UIViewController'a. Есть подозрение, что именно на него ориентируется стандартная Transition Animation и именно по этому так устанавливает фрейм. В этом коде она равна 0 (т.е. от низу экрана до начала вьюшки 0 пикселей) и наверное по этому firsViewController заходит за bottom bar.

[deleted-mezhevikin]

да, вы оказались правы, спасибо за помощь. самое интересное, что я паралельно искал решение поменять уродливую анимацию переходов в ios7, и вот решение - http://toster.ru/q/49522

Answer 2

[gooddy]

diff --git a/AKTabBarController/AKTabBarController.m b/AKTabBarController/AKTabBarController.m
index e8226ac..9853490 100755
--- a/AKTabBarController/AKTabBarController.m
+++ b/AKTabBarController/AKTabBarController.m
@@ -251,6 +251,9 @@ typedef enum {
     tabBar.hidden = NO;
     tabBar.transform = CGAffineTransformMakeTranslation(CGRectGetWidth(self.view.bounds) * directionVector, 0);
     // when the tabbarview is resized we can see the view behind
+	CGRect tmpTabBarView = tabBarView.contentView.frame;
+	tmpTabBarView.size.height = CGRectGetHeight(tabBarView.bounds) - CGRectGetHeight(tabBar.bounds);
+	tabBarView.contentView.frame = tmpTabBarView;
     
     [UIView animateWithDuration:((animated) ? kPushAnimationDuration : 0) animations:^{
         tabBar.transform = CGAffineTransformIdentity;

сохраняете в файлик fix.patch, заходите в репу компонента и

git apply /path/to/fix.patch

ну или руками добавить

Comments

[deleted-mezhevikin]

спасибо за ответ, проблема со скроллингом решается но таббар появляется странно http://www.youtube.com/watch?v=SbXHmMkQ8xE&feature=youtu.be Так лучше: <code> -(void) navigationController:(UINavigationController *)navigationController didShowViewController:(UIViewController *)viewController animated:(BOOL)animated{ CGRect viewControllerFrame = viewController.view.frame; viewControllerFrame.size.height = navigationController.view.frame.size.height-viewControllerFrame.origin.y; viewController.view.frame = viewControllerFrame; } </code>

Answer 3

[gleb_kudr]

Проблема связана с изменением положения контента относительно экрана в ios7. Раньше контент был под навигейшеном и над таббаром, а нижний и верхний оффсеты =0 (величина, на сколько верхнее и нижнее значение скролла отступает от границы контрола). В ios7 контент совпадает с размерами экрана, а оффсеты не равны нулю и определяются по хитрому и кривому механизму, который худо-бедно работает на нативных контролах, а с кастомными приходится их выставлять руками. В общем, гуглите «ios7 uitableview offset».

Comments

[deleted-mezhevikin]

спасибо, погуглил. советуют:

if ([[[UIDevice currentDevice] systemVersion] floatValue] >= 7) {
            self.edgesForExtendedLayout = UIRectEdgeNone;
            self.extendedLayoutIncludesOpaqueBars = NO;
            self.automaticallyAdjustsScrollViewInsets = NO;
        }

но это не работает для меня

[gleb_kudr]

Это не работает потому что вероятно автоматическое определение не подоходит для кастомного бара. Выставляйте UITableView.contentOffset руками.