ngOfficeUIFabric / ng-officeuifabric

Office UI Fabric (https://github.com/OfficeDev/office-ui-fabric) implementation for Angular
http://ngOfficeUiFabric.com
MIT License
321 stars 67 forks source link

Adding ng-controller to a ng-officeuifabric directive breaks it. Wrong design? #380

Closed ladaltamirano closed 8 years ago

ladaltamirano commented 8 years ago

Testing on

   "dependencies": {
      "angular": "1.5.5",
      "angular-animate": "1.5.5",
      "angular-sanitize": "1.5.5",
      "ng-office-ui-fabric": "0.12.0",
      "office-ui-fabric": "2.6.1"
   }

And running

     "name": "ngofficeuifabric.github.io"
     "version": "0.12.0"

Steps to Reproduce Behavior

I added an ng-controller to the <uif-message-bar> tag at /demos/uifMessageBar/ and it breaks it

Expected Behavior

This shouldn't happend at all. It should be able to work right straightforward. Wrong design?

Actual Behavior

Outputs code: {{ name }}

andrewconnell commented 8 years ago

Can you create a plunker showing this repo'd, just like the template mentioned when you created this issue? If all you did was what you describe in the issue, that's not going to be enough as you're also going to need to create a controller and attach it by name.

andrewconnell commented 8 years ago

Closing due to no response...