Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Commit f023ce7

Browse files
committed
refactor(menu): removed deprecated $mdOpenMenu API
- fixed test that used this old API BREAKING CHANGE: Removed the deprecated `$mdOpenMenu` API. It was deprecated in favor of `$mdMenu.open`.
1 parent 6d17e6e commit f023ce7

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

src/components/menu/js/menuController.js

-6
Original file line numberDiff line numberDiff line change
@@ -238,10 +238,4 @@ function MenuController($mdMenu, $attrs, $element, $scope, $mdUtil, $timeout, $r
238238
open: this.open,
239239
close: this.close
240240
};
241-
242-
// Deprecated APIs
243-
$scope.$mdOpenMenu = angular.bind(this, function() {
244-
$log.warn('mdMenu: The $mdOpenMenu method is deprecated. Please use `$mdMenu.open`.');
245-
return this.open.apply(this, arguments);
246-
});
247241
}

src/components/menu/menu.spec.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ describe('material.components.menu', function() {
198198
it('should focus on first item automatically', inject(function($compile, $rootScope, $document) {
199199
var menu = $compile(
200200
'<md-menu>' +
201-
'<button ng-click="$mdOpenMenu($event)">Hello World</button>' +
201+
'<button ng-click="$mdMenu.open($event)">Hello World</button>' +
202202
'<md-menu-content>' +
203203
'<md-menu-item>' +
204204
'<button id="menuItem0" ng-click="doSomething($event)"></button>' +
@@ -220,7 +220,7 @@ describe('material.components.menu', function() {
220220
it('should focus on first non-disabled item', inject(function($compile, $rootScope, $document) {
221221
var menu = $compile(
222222
'<md-menu>' +
223-
'<button ng-click="$mdOpenMenu($event)">Hello World</button>' +
223+
'<button ng-click="$mdMenu.open($event)">Hello World</button>' +
224224
'<md-menu-content>' +
225225
'<md-menu-item>' +
226226
'<button disabled ng-click="doSomething($event)"></button>' +

0 commit comments

Comments
 (0)