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-14215: Android ActionBar - handle errors gracefully #4453
Conversation
I think the real issue in the ticket is when navBarHidden is set to true, the activity.actionBar will return null. If this is a limitation of the action bar, we should document it. |
Thats part of the issue. "navBarHidden=true" is requesting no title bar in android. Action bar is part of the title bar, so there won't be an action bar. The other part is if user doesn't specify targetSDK to 11+, we would still be running into NPE. |
@@ -9,6 +9,9 @@ description: | | |||
To access the action bar, you must first open a heavyweight window or tab group that | |||
uses one of the action bar themes (such as the Android Holo theme). You can then | |||
access the action bar from the window or tab group's [activity](Titanium.Android.Activity). | |||
In addition, setting [Window.navBarHidden](Titanium.UI.Window.navBarHidden) or |
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.
In addition, setting the ...
Reviewed, ran validate and deploy. Left two minor notes. |
DR approved. |
Code reviewed and functionally tested. Accepted |
timob-14215: Android ActionBar - handle errors gracefully
Testing steps in https://jira.appcelerator.org/browse/TIMOB-14215