Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-25637] : Window toolbar animates when opening window #9698

Merged
merged 5 commits into from Jan 12, 2018

Conversation

vijaysingh-axway
Copy link
Contributor

@build
Copy link
Contributor

build commented Jan 3, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

@@ -956,6 +955,7 @@ - (void)setupWindowDecorations
SETPROP(@"translucent", setTranslucent);
SETPROP(@"tabBarHidden", setTabBarHidden);
SETPROPOBJ(@"toolbar", setToolbar);
[[controller navigationController] setToolbarHidden:!hasToolbar animated:YES];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seriously? Wow. Can we still ensure this doesn't cause side-effects for other use-cases like changing title-attributes etc?

@hansemannn
Copy link
Collaborator

Removing from QE-testing again as the initial author has discovered one outstanding issue.

@build build added the ios label Jan 9, 2018
@mukherjee2 mukherjee2 self-requested a review January 12, 2018 00:08
Copy link
Contributor

@mukherjee2 mukherjee2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passed FR. Details are in the backport version of the ticket.

@build build added the ios label Jan 12, 2018
@hansemannn hansemannn merged commit de032f0 into tidev:master Jan 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants