angular 5 EventEmitter give nothing

Question

Please, help me understand why EventEmitter in my example won't work. I can't see mistakes in my code, but still it won't work. I have no errors and no effect.

app.menu.ts (child component):

import { Component, Input, EventEmitter, Output } from '@angular/core';

@Component({
  selector: 'Menu',
  templateUrl: app.menu.html,
  styleUrls: ['app.menu.css'],
})

export class Menu {

  @Output() pick = new EventEmitter<any>();

  hidden: boolean = true;
  items: object[] = [
    {action: 'login', text: 'zaloguj się'},
    {action: 'halls', text: 'hale sportowe'},
    {action: 'sweemingPools', text: 'baseny'},
  ];

  constructor() {}

  toggler() {
    this.hidden = !this.hidden;
  };

  displayMarkers(type:string){
    console.log('test1', type); //work correctly
    this.pick.emit(type); //won't work
  };

}

child template:

  <div>
   <button (click)="toggler()">MENU</button>
   <div id="left-menu" [hidden]="hidden">
    <ul>
      <li *ngFor="let item of items"><a (click)=displayMarkers(item.action)>{{ item.text }}</a></li>
    </ul>
  </div>
 </div>

app.ts (parent component):

import { Component } from '@angular/core';

import { Menu } from './menu/app.menu';
import { Map } from './map/app.map';

@Component({
  selector: 'app-root',
  templateUrl: 'app.html',
  styleUrls: ['app.css'],
})
export class App {
  title: string = 'My webapp';

  onPick(event:string){
    console.log('test2', event); //nothing, won't work
  }
}

parent template:

  <div>
  <Menu (pick)="onPick($event)"></Menu>
  <div style="text-align:center">
    <h1>{{ title }}!</h1>
  </div>
  <Map></Map>
</div>

This is really simple code and I saw several similar examples. So it should work. And please, don't send me to different topicks, because I saw they really lot. I think onPick function never run, but why? What is wrong?

try putting double quotes around your click assignment: ``, this is mandatory. You are also using `templateUrl` instead of `template`. — Poul Kruijt, Mar 21 '18 at 13:04
unfortunately, still I nothing. I have write down in my console only "test1 ...", but "test2" no — Eva, Mar 21 '18 at 13:06
try to change onPick(event:string) to onPick(event:any) and check test2 — ElasticCode, Mar 21 '18 at 13:27

Johnny · Accepted Answer · 2018-03-21T14:04:46.680

Add your Menu Component to the app.module.ts declarations property like so:

import {NgModule} from '@angular/core'
import {BrowserModule} from '@angular/platform-browser'
import {AppComponent} from './app.component';
import { ChildComponent } from './app-child-component';
import {Menu} from './menu.component.ts';

@NgModule({
  imports: [ BrowserModule ],
  declarations: [ AppComponent, ChildComponent, Menu ],
  bootstrap: [ AppComponent ]
})
export class AppModule {}

Then in your parent component:

import {Component} from '@angular/core'

@Component({
  selector: 'app-child-component',
  template: `
    <Menu (pick)=onPick($event)></Menu>
  `,
})
export class ChildComponent {
  onPick(e) { console.log('picked'); alert('picked')};
}

and your component:

import { Component, Input, EventEmitter, Output } from '@angular/core';

@Component({
  selector: 'Menu',
  template: `
  <div>
  test
   <button (click)="toggler()">MENU</button>
   <div id="left-menu" [hidden]="hidden">
    <ul>
      <li *ngFor="let item of items"><a (click)="displayMarkers(item.action)">{{ item.text }}</a></li>
    </ul>
  </div>
 </div>`
})

export class Menu {

  @Output() pick = new EventEmitter<any>();

  hidden: boolean = true;
  items: object[] = [
    {action: 'login', text: 'zaloguj się'},
    {action: 'halls', text: 'hale sportowe'},
    {action: 'sweemingPools', text: 'baseny'},
  ];

  constructor() {}

  toggler() {
    this.hidden = !this.hidden;
  };

  displayMarkers(type:string){
    console.log('test1', type); //work correctly
    this.pick.emit(type); //won't work
  };

}

check this plunkr: https://plnkr.co/edit/TiKKLffi92KLsR7PYS5J?p=preview — Johnny, Mar 21 '18 at 14:01
I don't think, there's a problem. Manu (app.menu.ts) is just my child component, and work fine insteade of EventEmitter... but maybe you have right and I have some mistake in module.ts — Eva, Mar 21 '18 at 14:04
yes !!!! I bootstrapping whole my components, parent and children.... When I bootstrapping only parent component, everything work :-). Thank you people from great help and yours time — Eva, Mar 21 '18 at 14:08

angular 5 EventEmitter give nothing

1 Answers1