fix: Various minor bugfixes

src/modules/modal/components/modal.ts

@@ -28,6 +28,7 @@ import { ModalConfig, ModalSize } from "../classes/modal-config";
          [class.scroll]="mustScroll"
          [class.inverted]="isInverted"
          [ngClass]="dynamicClasses"
+         (click)="onClick($event)"
          #modal>
 
         <!-- Configurable close icon -->
@@ -282,6 +283,11 @@ export class SuiModal<T, U> implements OnInit, AfterViewInit {
         }
     }
 
+    public onClick(e:MouseEvent):void {
+        // Makes sense here, as the modal shouldn't be attached to any DOM element.
+        e.stopPropagation();
+    }
+
     @HostListener("document:keyup", ["$event"])
     public onDocumentKeyup(e:KeyboardEvent):void {
         if (e.keyCode === KeyCode.Escape) {

src/modules/popup/classes/popup-controller.ts

@@ -167,7 +167,6 @@ export abstract class SuiPopupController implements IPopup, OnDestroy {
 
     @HostListener("focusout", ["$event"])
     private onFocusOut(e:any):void {
-        console.log(e.relatedTarget);
         if (!this._element.nativeElement.contains(e.relatedTarget) &&
             !this.popup.elementRef.nativeElement.contains(e.relatedTarget) &&
             this.popup.config.trigger === PopupTrigger.Focus) {

src/modules/popup/directives/popup.directive.ts

@@ -6,6 +6,8 @@ import { SuiPopupConfig } from "../services/popup.service";
 import { SuiPopupController } from "../classes/popup-controller";
 import { SuiPopupTemplateController, ITemplatePopupContext, ITemplatePopupConfig } from "../classes/popup-template-controller";
 
+const templateRef = TemplateRef;
+
 @Directive({
     selector: "[suiPopup]",
     exportAs: "suiPopup"

src/modules/select/classes/select-base.ts

@@ -1,6 +1,6 @@
 import {
     ViewChild, HostBinding, ElementRef, HostListener, Input, ContentChildren, QueryList,
-    AfterContentInit, TemplateRef, ViewContainerRef, ContentChild, EventEmitter, Output
+    AfterContentInit, TemplateRef, ViewContainerRef, ContentChild, EventEmitter, Output, OnDestroy
 } from "@angular/core";
 import { Subscription } from "rxjs/Subscription";
 import { DropdownService, SuiDropdownMenu } from "../../dropdown";
@@ -16,7 +16,7 @@ export interface IOptionContext<T> extends ITemplateRefContext<T> {
 
 // We use generic type T to specify the type of the options we are working with,
 // and U to specify the type of the property of the option used as the value.
-export abstract class SuiSelectBase<T, U> implements AfterContentInit {
+export abstract class SuiSelectBase<T, U> implements AfterContentInit, OnDestroy {
     public dropdownService:DropdownService;
     public searchService:SearchService<T, U>;
 
@@ -322,7 +322,7 @@ export abstract class SuiSelectBase<T, U> implements AfterContentInit {
             this.drawTemplate(option.templateSibling, option.value);
         }
 
-        option.changeDetector.detectChanges();
+        option.changeDetector.markForCheck();
     }
 
     public abstract selectOption(option:T):void;
@@ -418,4 +418,8 @@ export abstract class SuiSelectBase<T, U> implements AfterContentInit {
             query: this.query
         });
     }
+
+    public ngOnDestroy():void {
+        this._renderedSubscriptions.forEach(s => s.unsubscribe());
+    }
 }