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
[ALOY-1584] Android: Ti.UI.Toolbar has no click event #853
Conversation
@@ -16,7 +16,7 @@ exports.parse = function(node, state) { | |||
code = '', | |||
xmlStyles = {}; | |||
// Properties inherited from ActionBar when Toolbar is passed to the Activity | |||
var inheritedProperties = ['backgroundImage', 'displayHomeAsUp', 'homeButtonEnabled', 'navigationMode', 'onHomeItemSelected']; | |||
var inheritedProperties = [CONST.BACKGROUND_IMAGE, CONST.DISPLAY_HOME_AS_UP, CONST.HOME_BUTTON_ENABLED, CONST.NAVIGATION_MODE, CONST.ON_HOME_ICON_ITEM_SELECTED]; |
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.
Please define CONST
.
@ypbnv, please add a test app for this. |
@feons Updated PR. I am not sure if these two files would be sufficient as test app, so if something else is needed - let me know. |
@ypbnv, in your Alloy repo folder you can use the command |
@ypbnv, please move |
Tests passed locally. Approved. |
JIRA: https://jira.appcelerator.org/browse/ALOY-1584
Description:
'onHomeIconItemsSelected ' was not properly inherited from ActionBar when used from Toolbar.
Properties of ActionBar shared between both components are now extracted as constants to avoid future mismatching.
Test case:
In JIRA ticket.