fix(list): wrapping secondary if it has ng-click

Secondary was being wrapped only if `md-checkbox` or `md-switch` were near by or `ng-click` was applied on the list item.

fixes #3928

src/components/list/list.js

@@ -110,6 +110,7 @@ function mdListItemDirective($mdAria, $mdConstant, $mdUtil, $timeout) {
  tEl.addClass('md-no-proxy');
  }
  }
+ wrapSecondary();
  setupToggleAria();
 
 
@@ -142,7 +143,9 @@ function mdListItemDirective($mdAria, $mdConstant, $mdUtil, $timeout) {
 
  tEl[0].setAttribute('tabindex', '-1');
  tEl.append(container);
+ }
 
+ function wrapSecondary() {
  if (secondaryItem && !isButton(secondaryItem) && secondaryItem.hasAttribute('ng-click')) {
  $mdAria.expect(secondaryItem, 'aria-label');
  var buttonWrapper = angular.element('<md-button class="md-secondary-container md-icon-button">');
@@ -155,10 +158,10 @@ function mdListItemDirective($mdAria, $mdConstant, $mdUtil, $timeout) {
 
  // Check for a secondary item and move it outside
  if ( secondaryItem && (
- secondaryItem.hasAttribute('ng-click') ||
+  secondaryItem.hasAttribute('ng-click') ||
  ( tAttrs.ngClick &&
-  isProxiedElement(secondaryItem) )
- )) {
+ isProxiedElement(secondaryItem) )
+  )) {
  tEl.addClass('md-with-secondary');
  tEl.append(secondaryItem);
  }