Solved

AngularJS directive issue

Posted on 2016-09-15

1 solution

Medium Priority

98 Views

Last Modified: 2016-09-17

The code below does not display my images. It works for a similar file but this its giving me fits. I believe I am not using the directive. properly. please let me know if I need to provide more info. In my editor I see the image for file so the image location is recognized by html page.

<!DOCTYPE html>
<html lang="en" ng-app="confusionApp">

<head>
     <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <!-- The above 3 meta tags *must* come first in the head; any other head
         content must come *after* these tags -->
    <title>Ristorante Con Fusion: Menu</title>
        <!-- Bootstrap -->
    <link href="../bower_components/bootstrap/dist/css/bootstrap.min.css" rel="stylesheet">
    <link href="../bower_components/bootstrap/dist/css/bootstrap-theme.min.css" rel="stylesheet">
    <link href="../bower_components/font-awesome/css/font-awesome.min.css" rel="stylesheet">
    <link href="styles/bootstrap-social.css" rel="stylesheet">
    <link href="styles/mystyles.css" rel="stylesheet">

    <!-- HTML5 shim and Respond.js for IE8 support of HTML5 elements and media queries -->
    <!-- WARNING: Respond.js doesn't work if you view the page via file:// -->
    <!--[if lt IE 9]>
      <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
      <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
    <![endif]-->
</head>

<body>

    <div class="container">
        
    <div class="row row-content" ng-controller="dishDetailController as dishCtrl">
            <li class="media" ng-model="dishc in dishCtrl.dish" > <!--| filter:menuCtrl.filtText">-->

                    
                <div class="media-left media-middle">
                        <a href="#">
                        <img class="media-object img-thumbnail"
                         ng-src={{dishc.image}} alt="Uthappizza">
                        </a>
                    </div>
                    <div class="media-body">
                        <h2 class="media-heading">{{dishc.name}}
                         <span class="label label-danger">{{dishc.label}}</span>
                         <span class="badge">{{dishc.price | currency}}</span></h2>
                         <p>{{dishc.description}}</p>

                    </div>
                </li>    

                </div>
            </div>
            <div class="col-xs-9 col-xs-offset-1">
                <p>Put the comments here</p>
            </div>
        

    <script src="../bower_components/angular/angular.min.js"></script>
    <script>

        var app = angular.module('confusionApp',[]);
        app.controller('dishDetailController', function() {

            var dish={
                          name:'Uthapizza',
                          image: 'images/uthapizza.png',
                          category: 'mains', 
                          label:'Hot',
                          price:'4.99',
                          description:'A unique combination of Indian Uthappam (pancake) and Italian pizza, topped with Cerignola olives, ripe vine cherry tomatoes, Vidalia onion, Guntur chillies and Buffalo Paneer.',
                           comments: [
                               {
                                   rating:5,
                                   comment:"Imagine all the eatables, living in conFusion!",
                                   author:"John Lemon",
                                   date:"2012-10-16T17:57:28.556094Z"
                               },
                               {
                                   rating:4,
                                   comment:"Sends anyone to heaven, I wish I could get my mother-in-law to eat it!",
                                   author:"Paul McVites",
                                   date:"2014-09-05T17:57:28.556094Z"
                               },
                               {
                                   rating:3,
                                   comment:"Eat it, just eat it!",
                                   author:"Michael Jaikishan",
                                   date:"2015-02-13T17:57:28.556094Z"
                               },
                               {
                                   rating:4,
                                   comment:"Ultimate, Reaching for the stars!",
                                   author:"Ringo Starry",
                                   date:"2013-12-02T17:57:28.556094Z"
                               },
                               {
                                   rating:2,
                                   comment:"It's your birthday, we're gonna party!",
                                   author:"25 Cent",
                                   date:"2011-12-02T17:57:28.556094Z"
                               }
                               
                           ]
                    };
            
            this.dish = dish;
            
        });

    </script>

</body>

</html>

Open in new window

Screen-Shot-2016-09-16-at-1.02.43-AM.png

Comment

Question by:Leo Torres

[X]

Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

Help others & share knowledge
Earn cash & points
Learn & ask questions

Join The Community

6 Comments

LVL 58

Expert Comment

by:Julian Hansen

ID: 418010242016-09-15

I believe line 31 is the problem

<li class="media" ng-model="dishc in dishCtrl.dish" >

Open in new window

You are using an ng-model instead of ng-repeat. Should be

<li class="media" ng-repeat="dishc in dishCtrl.dish" >

Open in new window

LVL 8

Author Comment

by:Leo Torres

ID: 418013392016-09-16

I tried ng-repeat and the result was worst. Here is my result

Also i only have 1 dish item as you can see below in the code.
here is what the final product should look like.

LVL 58

Accepted Solution

by:Julian Hansen

Julian Hansen earned 2000 total points

ID: 418014292016-09-16

Ok - that is helpful - you will need to help us here we don't know what you are aiming at.

What was your intention here?

<li class="media" ng-model="dishc in dishCtrl.dish" >

Open in new window

You cannot use that expression for ng-model?

There are a number of things that you need to fix with the code

1. The ng-model - you don't need this on the <li> I see what you were trying to do but that is not the correct way - see below for an explanation

2. Your comments are outside of the controller block - which means you won't be able to access the comments - you need to put that inside the controller block.

Let's break your code down.
You have elected to use the controller as approach instead of scope - which is good but here is how it works.

In your controller you have the structure called dish. You are making this a controller property by this line
(A)

this.dish = dish;

Open in new window

In your html you have

<div class="row row-content" ng-controller="dishDetailController as dishCtrl">

Open in new window

That means to access your model you need to use the controller alias in this case dishCtrl

Going back to the code snippet (A) your controller representation of your data is dish - therefore to output the values of your model you need to reference them as

dishCtrl.dish.<property>

Open in new window

So for the image this would be

dishCtrl.dish.image

Open in new window

In your comments section you would use an ng-repeat - I updated your code

<body>
<div class="container">
  <div class="row row-content" ng-controller="dishDetailController as dishCtrl">
    <li class="media"> <!--| filter:menuCtrl.filtText">-->
      <div class="media-left media-middle">
        <a href="#">
          <img class="media-object img-thumbnail"
          ng-src={{dishCtrl.dish.image}} alt="Uthappizza">
        </a>
      </div>
      <div class="media-body">
        <h2 class="media-heading">{{dishCtrl.dish.name}}
        <span class="label label-danger">{{dishCtrl.dish.label}}</span>
        <span class="badge">{{dishCtrl.dish.price | currency}}</span></h2>
        <p>{{dishCtrl.dish.description}}</p>

      </div>
    </li>    

    <div class="col-xs-9 col-xs-offset-1">
      <ul>
        <li ng-repeat="comment in dishCtrl.dish.comments">
          <h3>{{comment.rating}} stars</h3>
          <div>{{comment.comment}}</div>
          <div>&dash; {{comment.author}}, {{comment.date | date}}</div>
        </li>
      </ul>
    </div>
  </div>
</div>

<script src="../bower_components/angular/angular.min.js"></script>
<script>
var app = angular.module('confusionApp',[]);
app.controller('dishDetailController', function() {

  var dish={
    name:'Uthapizza',
    image: 'images/foreground1.png',
    category: 'mains', 
    label:'Hot',
    price:'4.99',
    description:'A unique combination of Indian Uthappam (pancake) and Italian pizza, topped with Cerignola olives, ripe vine cherry tomatoes, Vidalia onion, Guntur chillies and Buffalo Paneer.',
    comments: [{
        rating:5,
        comment:"Imagine all the eatables, living in conFusion!",
        author:"John Lemon",
        date:"2012-10-16T17:57:28.556094Z"
      },
      {
        rating:4,
        comment:"Sends anyone to heaven, I wish I could get my mother-in-law to eat it!",
        author:"Paul McVites",
        date:"2014-09-05T17:57:28.556094Z"
      },
      {
        rating:3,
        comment:"Eat it, just eat it!",
        author:"Michael Jaikishan",
        date:"2015-02-13T17:57:28.556094Z"
      },
      {
        rating:4,
        comment:"Ultimate, Reaching for the stars!",
        author:"Ringo Starry",
        date:"2013-12-02T17:57:28.556094Z"
      },
      {
        rating:2,
        comment:"It's your birthday, we're gonna party!",
        author:"25 Cent",
        date:"2011-12-02T17:57:28.556094Z"
      }
    ]
  };

  this.dish = dish;

});
</script>
</body>

Open in new window

you can see it working here

LVL 8

Author Comment

by:Leo Torres

ID: 418025932016-09-16

Thank you for being an excellent resource. I was using ng-model just because I thought it was the right directive.

LVL 8

Author Closing Comment

by:Leo Torres

ID: 418025942016-09-16

Thanks again!

LVL 58

Expert Comment

by:Julian Hansen

ID: 418026782016-09-17

ng-model is for binding a view element to your model - i.e. elements that have a value.

To access the view for output or expressions you either use the controllerAs prefix to dereference the property you want in your controller - or if you are not using controller as (scope) then you can refer to the property directly.

Glad to be of help and you are most welcome.

Featured Post

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Beginner's Guide to HTML and CSS

Article by: Experts Exchange

Building a website can seem like a daunting task to the uninitiated but it really only requires knowledge of two basic languages: HTML and CSS.

4 Methods for creating image overlays in HTML using CSS and PHP GD2

Article by: Julian

This article discusses four methods for overlaying images in a container on a web page

Introduction to CSS (Part 2)

Video by: Jason

The viewer will learn the benefit of using external CSS files and the relationship between class and ID selectors. Create your external css file by saving it as style.css then set up your style tags: (CODE) Reference the nav tag and set your prop…