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-6068 remove hideTabBar from KS Nook #784
Conversation
@@ -33,10 +33,6 @@ tableview.addEventListener('click', function(e) | |||
title:e.rowData.title | |||
}); | |||
|
|||
if (e.index == 3) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be an iOS only check instead of code deletion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could, but I figured since this is nook-only, I'd just delete it. But I can change if you want.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair point -- There isn't a Nook iOS that I know of :) sounds good to me, let's leave it
Code reviewed. Request accepted. |
Pull accepted Tested on Nook, verified behavior of window (standalone) not producing crash |
Code and functional review passed, Request accepted |
TIMOB-6068 remove hideTabBar from KS Nook
http://jira.appcelerator.org/browse/TIMOB-6068
Test is the failcase in the app (you should be able to run "Window Standalone" without error).
See description of this "fix".