Merge pull request #446 from cal-smith/issue#46

fix(combobox): tidy up types

Author: zvonimirfras

src/combobox/combobox.component.ts

@@ -123,8 +123,6 @@ export class ComboBox implements OnChanges, OnInit, AfterViewInit, AfterContentI
 	 * ];
 	 * ```
 	 *
-	 * @type {Array<ListItem>}
-	 * @memberof ComboBox
 	 */
 	@Input() items: Array<ListItem> = [];
 	/**
@@ -154,7 +152,7 @@ export class ComboBox implements OnChanges, OnInit, AfterViewInit, AfterContentI
 	 * }
 	 * ```
 	 */
-	@Output() selected: EventEmitter<ListItem> = new EventEmitter<ListItem>();
+	@Output() selected = new EventEmitter<ListItem | ListItem[]>();
 	/**
 	 * Bubbles from `n-pill-input` when the user types a value and presses enter. Intended to be used to add items to the list.
 	 *
@@ -205,17 +203,13 @@ export class ComboBox implements OnChanges, OnInit, AfterViewInit, AfterContentI
 
 	/**
 	 * Creates an instance of ComboBox.
-	 * @param {ElementRef} elementRef
-	 * @memberof ComboBox
 	 */
 	constructor(protected elementRef: ElementRef) {}
 
 	/**
 	 * Lifecycle hook.
 	 * Updates pills if necessary.
 	 *
-	 * @param {any} changes
-	 * @memberof ComboBox
 	 */
 	ngOnChanges(changes) {
 		if (changes.items) {
@@ -285,7 +279,6 @@ export class ComboBox implements OnChanges, OnInit, AfterViewInit, AfterContentI
 
 	/**
 	 * Handles `Escape` key closing the dropdown, and arrow up/down focus to/from the dropdown list.
-	 * @param {KeyboardEvent} ev
 	 */
 	@HostListener("keydown", ["$event"])
 	hostkeys(ev: KeyboardEvent) {
@@ -379,7 +372,6 @@ export class ComboBox implements OnChanges, OnInit, AfterViewInit, AfterContentI
 
 	/**
 	 * Sets the list group filter, and manages single select item selection.
-	 * @param {string} searchString
 	 */
 	public onSearch(searchString) {
 		this.view.filterBy(searchString);

src/dropdown/abstract-dropdown-view.class.ts

@@ -16,9 +16,11 @@ export class AbstractDropdownView {
 
 	get items(): Array<ListItem> | Observable<Array<ListItem>> { return; }
 	/**
-	 * Emits selection events to other class.
+	 * Emits selection events to controlling classes
+	 *
+	 * Deprecated: `Object` as a valid type.
 	 */
-	@Output() select: EventEmitter<Object>;
+	@Output() select: EventEmitter<{item: ListItem } | ListItem[] | Object>;
 	/**
 	 * Event to suggest a blur on the view.
 	 * Emits _after_ the first/last item has been focused.