Skip to content

Loading…

Menu's scroller should be isChrome: true #83

Merged
merged 1 commit into from

2 participants

@mbessey

Otherwise, you can't rebuild the menu with new items via
destroyClientControls() and createComponent()

This is fallout from the fix for ENYO-689

Mark Bessey Menu's scroller should be isChrome: true
Otherwise, you can't rebuild the menu with new items via
destroyClientControls() and createComponent()
85bba67
@unwiredben unwiredben merged commit b28489a into master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 28, 2012
  1. Menu's scroller should be isChrome: true

    Mark Bessey committed
    Otherwise, you can't rebuild the menu with new items via
    destroyClientControls() and createComponent()
Showing with 1 addition and 1 deletion.
  1. +1 −1 source/Menu.js
View
2 source/Menu.js
@@ -56,7 +56,7 @@ enyo.kind({
this.maxHeightChanged();
},
initComponents: function() {
- this.scrolling ? this.createComponents(this.childComponents) : enyo.nop;
+ this.scrolling ? this.createComponents(this.childComponents, {isChrome: true}) : enyo.nop;
this.inherited(arguments);
},
getScroller: function() {
Something went wrong with that request. Please try again.