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-25334] iOS: Prevent Ti.UI.TableView & Ti.UI.ListView from sliding up in navigation-window #9470

Merged
merged 6 commits into from Sep 26, 2017

Conversation

hansemannn
Copy link
Collaborator

@build
Copy link
Contributor

build commented Sep 25, 2017

Warnings
⚠️

🔬 There are library changes, but no changes to the unit tests. That's OK as long as you're refactoring existing code

Generated by 🚫 dangerJS

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR passed.

@ewieberappc
Copy link
Contributor

FR Passed. ListViews and TableViews do not slide when opened from or opening a nav window. Tested using the provided sample code and KitchenSink v2.

@hansemannn hansemannn merged commit cb9ad51 into tidev:master Sep 26, 2017
rlustemberg added a commit to rlustemberg/titanium_mobile that referenced this pull request Sep 27, 2017
* commit 'ac2eccd028534a485fc1887696753097b488e35b': (721 commits)
  [TIMOB-25347] Fix Ti.UI.Android.DrawerLayout layout (tidev#9477)
  titanium_ui_drawer_layout.xml fixes: background, elevation and fitsSystemWindows (tidev#9439)
  [TIMOB-25334] iOS: Prevent Ti.UI.TableView & Ti.UI.ListView from sliding up in navigation-window (tidev#9470)
  [TIMOB-25318] Fixed bug where sim builds were blowing up when incorrectly trying to access the provisioning profile. (tidev#9473)
  [TIMOB-25318] Improved team-id auto-selection and allow the team-id t… (tidev#9468)
  Fix typo in dangerfile for handling test failures
  [TIMOB-25332] Fix table-view estimated heights
  First try at reporting failed tests in the Danger.JS PR comment.
  Fix clang-format warnings
  Wipe the workspace of the node running S3 publishing (Deploy stage) afterwards. Also wipe the dist/ dir before unarchiving. This should prevent retaining old build artifacts.
  [TIMOB-25318] Fixed typos, consolidated regexp, and improved method name.
  [TIMOB-25318] Improved provisioning profile validation to check against the cert.
  Ignore junit reports in git
  If first build on a mainline branch, trigger Windows SDK build after done
  Wrap clone of test suite into try/catch block and explicitly warn if we fail that most likely cause is that the target branch doesn't exist.
  Fix list of files stashes for danger.js
  Try to fix post-build Danger stage. Need to stash/unstash files since we likely have changed nodes
  Move running Danger.JS to last stage for PRs only. Add more danger checks.
  Fix eslint warnings
  Don't pre-set build status to 'SUCCESS' - causes us to mark PR as good when git checkout step is done, which is BAD
  ...

# Conflicts:
#	iphone/Classes/TiApp.h
#	iphone/Classes/TiApp.m
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