fixes

Author: kara

e2e/components/menu/menu-page.ts

@@ -14,11 +14,9 @@ export class MenuPage {
 
   body() { return element(by.tagName('body')); }
 
-  firstItem() { return element(by.id('one')); }
-
-  secondItem() { return element(by.id('two')); }
-
-  thirdItem() { return element(by.id('three')); }
+  items(index: number) {
+    return element.all(by.css('[md-menu-item]')).get(index);
+  }
 
   textArea() { return element(by.id('text')); }
 

e2e/components/menu/menu.e2e.ts

@@ -23,23 +23,23 @@ describe('menu', function () {
 
   it('should close menu when menu item is clicked', function () {
     page.trigger().click();
-    page.firstItem().click();
+    page.items(0).click();
     page.expectMenuPresent(false);
   });
 
   it('should run click handlers on regular menu items', function() {
     page.trigger().click();
-    page.firstItem().click();
+    page.items(0).click();
     expect(page.getResultText()).toEqual('one');
 
     page.trigger().click();
-    page.secondItem().click();
+    page.items(1).click();
     expect(page.getResultText()).toEqual('two');
   });
 
   it('should run not run click handlers on disabled menu items', function() {
     page.trigger().click();
-    page.thirdItem().click();
+    page.items(2).click();
     expect(page.getResultText()).toEqual('');
   });
 
@@ -89,8 +89,13 @@ describe('menu', function () {
 
     it('should align overlay bottom to origin bottom when y-position is "above"', () => {
       page.aboveTrigger().click();
+      page.aboveTrigger().getCssValue('height').then((height) => console.log('trigger height:', height));
+      page.aboveTrigger().getCssValue('padding').then((padding) => console.log('trigger padding:', padding));
+      page.aboveMenu().getCssValue('height').then((height) => console.log('menu height: ', height))
+      page.aboveMenu().getCssValue('padding').then((padding) => console.log('menu padding: ', padding))
       page.aboveTrigger().getLocation().then((trigger) => {
-
+        console.log("trigger.y: ", trigger.y);
+        page.aboveMenu().getLocation().then((location) => console.log('menu y', location.y))
         // the menu's bottom corner must be attached to the trigger's bottom corner.
         // menu.x should equal trigger.x because only y position has changed.
         // menu = 64px high. trigger = 20px high. 64 - 20 = 44px of menu extending up past trigger.

src/components/menu/menu-positions.ts

@@ -0,0 +1,4 @@
+
+export type MenuPositionX = 'before' | 'after';
+
+export type MenuPositionY = 'above' | 'below';

src/components/menu/menu.scss

@@ -23,7 +23,7 @@ $md-menu-vertical-padding: 8px !default;
 
   // max height must be 100% of the viewport height + one row height
   max-height: calc(100vh + 48px);
-  overflow: scroll;
+  overflow: auto;
   -webkit-overflow-scrolling: touch;   // for momentum scroll on mobile
 
   background: md-color($md-background, 'card');

src/components/menu/menu.ts

@@ -6,11 +6,13 @@ import {
     Attribute,
     Component,
     EventEmitter,
+    Input,
     Output,
     TemplateRef,
     ViewChild,
     ViewEncapsulation
 } from '@angular/core';
+import {MenuPositionX, MenuPositionY} from './menu-positions';
 import {MdMenuInvalidPositionX, MdMenuInvalidPositionY} from './menu-errors';
 
 @Component({
@@ -25,52 +27,50 @@ import {MdMenuInvalidPositionX, MdMenuInvalidPositionY} from './menu-errors';
 export class MdMenu {
   private _showClickCatcher: boolean = false;
   private _classList: Object;
-  positionX: 'before' | 'after' = 'after';
-  positionY: 'above' | 'below' = 'below';
+  positionX: MenuPositionX = 'after';
+  positionY: MenuPositionY = 'below';
 
-  @Output() close = new EventEmitter;
   @ViewChild(TemplateRef) templateRef: TemplateRef<any>;
 
-  constructor(@Attribute('x-position') posX: 'before' | 'after',
-              @Attribute('y-position') posY: 'above' | 'below',
-              @Attribute('class') classes: string) {
+  constructor(@Attribute('x-position') posX: MenuPositionX,
+              @Attribute('y-position') posY: MenuPositionY) {
     if (posX) { this._setPositionX(posX); }
     if (posY) { this._setPositionY(posY); }
-    this._mirrorHostClasses(classes);
-  }
-
-  /**
-   * This function toggles the display of the menu's click catcher element.
-   * This element covers the viewport when the menu is open to detect clicks outside the menu.
-   * TODO: internal
-   */
-  _setClickCatcher(bool: boolean): void {
-    this._showClickCatcher = bool;
   }
 
   /**
    * This method takes classes set on the host md-menu element and applies them on the
    * menu template that displays in the overlay container.  Otherwise, it's difficult
    * to style the containing menu from outside the component.
-   * @param classes: list of class names
+   * @param classes list of class names
    */
-  private _mirrorHostClasses(classes: string): void {
-    if (!classes) { return; }
-
+  @Input('class')
+  set classList(classes: string) {
     this._classList = classes.split(' ').reduce((obj: any, className: string) => {
       obj[className] = true;
       return obj;
     }, {});
   }
 
-  private _setPositionX(pos: 'before' | 'after'): void {
+  @Output() close = new EventEmitter;
+
+  /**
+   * This function toggles the display of the menu's click catcher element.
+   * This element covers the viewport when the menu is open to detect clicks outside the menu.
+   * TODO: internal
+   */
+  _setClickCatcher(bool: boolean): void {
+    this._showClickCatcher = bool;
+  }
+
+  private _setPositionX(pos: MenuPositionX): void {
     if ( pos !== 'before' && pos !== 'after') {
       throw new MdMenuInvalidPositionX();
     }
     this.positionX = pos;
   }
 
-  private _setPositionY(pos: 'above' | 'below'): void {
+  private _setPositionY(pos: MenuPositionY): void {
     if ( pos !== 'above' && pos !== 'below') {
       throw new MdMenuInvalidPositionY();
     }