carbon-components-angular 5.47.1 → 5.47.2

package/dialog/dialog.directive.d.ts

@@ -37,6 +37,9 @@ export declare class DialogDirective implements OnInit, OnDestroy, OnChanges {
     protected elementRef: ElementRef;
     protected viewContainerRef: ViewContainerRef;
     protected dialogService: DialogService;
+    /**
+     * Deprecated as of v5
+     */
     protected eventService: EventService;
     static dialogCounter: number;
     /**
@@ -122,6 +125,7 @@ export declare class DialogDirective implements OnInit, OnDestroy, OnChanges {
      * Keeps a reference to the currently opened dialog
      */
     protected dialogRef: ComponentRef<Dialog>;
+    private subscriptions;
     /**
      * Creates an instance of DialogDirective.
      * @param elementRef
@@ -129,7 +133,11 @@ export declare class DialogDirective implements OnInit, OnDestroy, OnChanges {
      * @param dialogService
      * @param eventService
      */
-    constructor(elementRef: ElementRef, viewContainerRef: ViewContainerRef, dialogService: DialogService, eventService: EventService);
+    constructor(elementRef: ElementRef, viewContainerRef: ViewContainerRef, dialogService: DialogService, 
+    /**
+     * Deprecated as of v5
+     */
+    eventService: EventService);
     ngOnChanges(changes: SimpleChanges): void;
     /**
      * Sets the config object and binds events for hovering or clicking before

package/docs/documentation/components/Toggletip.html

@@ -137,6 +137,7 @@
     </p>
     <p class="comment">
                 <code>AfterViewInit</code>
+                <code>OnDestroy</code>
     </p>
 
 
@@ -209,6 +210,10 @@
                             <li>
                                 <a href="#documentClick" >documentClick</a>
                             </li>
+                            <li>
+                                    <span class="modifier">Private</span>
+                                <a href="#subscription" >subscription</a>
+                            </li>
                             <li>
                                     <span class="modifier"></span>
                                 <a href="#toggletipClass" >toggletipClass</a>
@@ -274,6 +279,9 @@
                             <li>
                                 <a href="#ngAfterViewInit" >ngAfterViewInit</a>
                             </li>
+                            <li>
+                                <a href="#ngOnDestroy" >ngOnDestroy</a>
+                            </li>
                             <li>
                                 <a href="#cleanUp" >cleanUp</a>
                             </li>
@@ -286,9 +294,6 @@
                             <li>
                                 <a href="#ngOnChanges" >ngOnChanges</a>
                             </li>
-                            <li>
-                                <a href="#ngOnDestroy" >ngOnDestroy</a>
-                            </li>
                             <li>
                                 <a href="#recomputePosition" >recomputePosition</a>
                             </li>
@@ -404,7 +409,7 @@
                 </tr>
                         <tr>
                             <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="47" class="link-to-prism">src/toggletip/toggletip.component.ts:47</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="49" class="link-to-prism">src/toggletip/toggletip.component.ts:49</a></div>
                             </td>
                         </tr>
 
@@ -501,7 +506,7 @@
                 </tr>
                         <tr>
                             <td class="col-md-2" colspan="2">
-                                    <div class="io-line">Defined in <a href="" data-line="40" class="link-to-prism">src/toggletip/toggletip.component.ts:40</a></div>
+                                    <div class="io-line">Defined in <a href="" data-line="41" class="link-to-prism">src/toggletip/toggletip.component.ts:41</a></div>
                             </td>
                         </tr>
             </tbody>
@@ -533,7 +538,7 @@
                             </tr>
                         <tr>
                             <td class="col-md-2" colspan="2">
-                                    <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:43</a></code>
+                                    <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:44</a></code>
 </div>
                             </td>
                         </tr>
@@ -863,7 +868,7 @@ This is not toggleable - should be assigned once</p>
                 </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="42" class="link-to-prism">src/toggletip/toggletip.component.ts:42</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="43" class="link-to-prism">src/toggletip/toggletip.component.ts:43</a></div>
                         </td>
                     </tr>
 
@@ -936,8 +941,8 @@ This is not toggleable - should be assigned once</p>
 
                     <tr>
                         <td class="col-md-4">
-                            <div class="io-line">Defined in <a href="" data-line="88"
-                                    class="link-to-prism">src/toggletip/toggletip.component.ts:88</a></div>
+                            <div class="io-line">Defined in <a href="" data-line="90"
+                                    class="link-to-prism">src/toggletip/toggletip.component.ts:90</a></div>
                         </td>
                     </tr>
 
@@ -972,8 +977,8 @@ This is not toggleable - should be assigned once</p>
 
                     <tr>
                         <td class="col-md-4">
-                            <div class="io-line">Defined in <a href="" data-line="101"
-                                    class="link-to-prism">src/toggletip/toggletip.component.ts:101</a></div>
+                            <div class="io-line">Defined in <a href="" data-line="107"
+                                    class="link-to-prism">src/toggletip/toggletip.component.ts:107</a></div>
                         </td>
                     </tr>
 
@@ -1056,8 +1061,8 @@ This is not toggleable - should be assigned once</p>
 
                     <tr>
                         <td class="col-md-4">
-                            <div class="io-line">Defined in <a href="" data-line="95"
-                                    class="link-to-prism">src/toggletip/toggletip.component.ts:95</a></div>
+                            <div class="io-line">Defined in <a href="" data-line="97"
+                                    class="link-to-prism">src/toggletip/toggletip.component.ts:97</a></div>
                         </td>
                     </tr>
 
@@ -1128,8 +1133,8 @@ This is not toggleable - should be assigned once</p>
 
                     <tr>
                         <td class="col-md-4">
-                            <div class="io-line">Defined in <a href="" data-line="88"
-                                    class="link-to-prism">src/toggletip/toggletip.component.ts:88</a></div>
+                            <div class="io-line">Defined in <a href="" data-line="90"
+                                    class="link-to-prism">src/toggletip/toggletip.component.ts:90</a></div>
                         </td>
                     </tr>
 
@@ -1201,7 +1206,50 @@ This is not toggleable - should be assigned once</p>
                     </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:60</a></code>
+                                <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:62</a></code>
+</div>
+                        </td>
+                    </tr>
+
+
+            <tr>
+                <td class="col-md-4">
+
+                        <div class="io-description">
+                            <b>Returns : </b>        <code><a href="https://www.typescriptlang.org/docs/handbook/basic-types.html" target="_blank" >void</a></code>
+
+                        </div>
+                </td>
+            </tr>
+        </tbody>
+    </table>
+    <table class="table table-sm table-bordered">
+        <tbody>
+            <tr>
+                <td class="col-md-4">
+                    <a name="ngOnDestroy"></a>
+                    <span class="name">
+                        <span ><b>ngOnDestroy</b></span>
+                        <a href="#ngOnDestroy"><span class="icon ion-ios-link"></span></a>
+                    </span>
+                </td>
+            </tr>
+            <tr>
+                <td class="col-md-4">
+<code>ngOnDestroy()</code>
+                </td>
+            </tr>
+
+
+                    <tr>
+                        <td class="col-md-4">
+                            <div class="io-line">Inherited from         <code><a href="../directives/PopoverContainer.html" target="_self" >PopoverContainer</a></code>
+</div>
+                        </td>
+                    </tr>
+                    <tr>
+                        <td class="col-md-4">
+                                <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:103</a></code>
 </div>
                         </td>
                     </tr>
@@ -1481,51 +1529,6 @@ This is not toggleable - should be assigned once</p>
             </tr>
         </tbody>
     </table>
-    <table class="table table-sm table-bordered">
-        <tbody>
-            <tr>
-                <td class="col-md-4">
-                    <a name="ngOnDestroy"></a>
-                    <span class="name">
-                        <span ><b>ngOnDestroy</b></span>
-                        <a href="#ngOnDestroy"><span class="icon ion-ios-link"></span></a>
-                    </span>
-                </td>
-            </tr>
-            <tr>
-                <td class="col-md-4">
-<code>ngOnDestroy()</code>
-                </td>
-            </tr>
-
-
-                    <tr>
-                        <td class="col-md-4">
-                            <div class="io-line">Inherited from         <code><a href="../directives/PopoverContainer.html" target="_self" >PopoverContainer</a></code>
-</div>
-                        </td>
-                    </tr>
-                    <tr>
-                        <td class="col-md-4">
-                                <div class="io-line">Defined in         <code><a href="../directives/PopoverContainer.html#source" target="_self" >PopoverContainer:278</a></code>
-</div>
-                        </td>
-                    </tr>
-
-
-            <tr>
-                <td class="col-md-4">
-                        <div class="io-description"><p>Clean up</p>
-</div>
-
-                        <div class="io-description">
-                            <b>Returns : </b>        <code><a href="https://www.typescriptlang.org/docs/handbook/basic-types.html" target="_blank" >void</a></code>
-
-                        </div>
-                </td>
-            </tr>
-        </tbody>
-    </table>
     <table class="table table-sm table-bordered">
         <tbody>
             <tr>
@@ -1762,7 +1765,7 @@ This is not toggleable - should be assigned once</p>
                 </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="45" class="link-to-prism">src/toggletip/toggletip.component.ts:45</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="46" class="link-to-prism">src/toggletip/toggletip.component.ts:46</a></div>
                         </td>
                     </tr>
 
@@ -1787,7 +1790,34 @@ This is not toggleable - should be assigned once</p>
                 </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="47" class="link-to-prism">src/toggletip/toggletip.component.ts:47</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="48" class="link-to-prism">src/toggletip/toggletip.component.ts:48</a></div>
+                        </td>
+                    </tr>
+
+
+        </tbody>
+    </table>
+    <table class="table table-sm table-bordered">
+        <tbody>
+            <tr>
+                <td class="col-md-4">
+                    <a name="subscription"></a>
+                    <span class="name">
+                            <span class="modifier">Private</span>
+                        <span ><b>subscription</b></span>
+                        <a href="#subscription"><span class="icon ion-ios-link"></span></a>
+                    </span>
+                </td>
+            </tr>
+                <tr>
+                    <td class="col-md-4">
+                        <i>Type : </i>    <code>Subscription</code>
+
+                    </td>
+                </tr>
+                    <tr>
+                        <td class="col-md-4">
+                                <div class="io-line">Defined in <a href="" data-line="49" class="link-to-prism">src/toggletip/toggletip.component.ts:49</a></div>
                         </td>
                     </tr>
 
@@ -1822,7 +1852,7 @@ This is not toggleable - should be assigned once</p>
                 </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="42" class="link-to-prism">src/toggletip/toggletip.component.ts:42</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="43" class="link-to-prism">src/toggletip/toggletip.component.ts:43</a></div>
                         </td>
                     </tr>
 
@@ -1854,7 +1884,7 @@ This is not toggleable - should be assigned once</p>
                 </tr>
                     <tr>
                         <td class="col-md-4">
-                                <div class="io-line">Defined in <a href="" data-line="38" class="link-to-prism">src/toggletip/toggletip.component.ts:38</a></div>
+                                <div class="io-line">Defined in <a href="" data-line="39" class="link-to-prism">src/toggletip/toggletip.component.ts:39</a></div>
                         </td>
                     </tr>
 
@@ -2167,9 +2197,10 @@ This is not toggleable - should be assigned once</p>
 	HostListener,
 	Input,
 	NgZone,
+	OnDestroy,
 	Renderer2
 } from &quot;@angular/core&quot;;
-import { fromEvent } from &quot;rxjs&quot;;
+import { fromEvent, Subscription } from &quot;rxjs&quot;;
 import { PopoverContainer } from &quot;carbon-components-angular/popover&quot;;
 import { ToggletipButton } from &quot;./toggletip-button.directive&quot;;
 
@@ -2192,7 +2223,7 @@ import { ToggletipButton } from &quot;./toggletip-button.directive&quot;;
 		&lt;/cds-popover-content&gt;
 	&#x60;
 })
-export class Toggletip extends PopoverContainer implements AfterViewInit {
+export class Toggletip extends PopoverContainer implements AfterViewInit, OnDestroy {
 	static toggletipCounter &#x3D; 0;
 
 	@Input() id &#x3D; &#x60;tooltip-${Toggletip.toggletipCounter++}&#x60;;
@@ -2203,6 +2234,7 @@ export class Toggletip extends PopoverContainer implements AfterViewInit {
 	@ContentChild(ToggletipButton, { read: ElementRef }) btn!: ElementRef;
 
 	documentClick &#x3D; this.handleFocusOut.bind(this);
+	private subscription: Subscription;
 
 	constructor(
 		protected hostElement: ElementRef,
@@ -2219,7 +2251,7 @@ export class Toggletip extends PopoverContainer implements AfterViewInit {
 		this.initializeReferences();
 
 		// Listen for click events on trigger
-		fromEvent(this.btn.nativeElement, &quot;click&quot;)
+		this.subscription &#x3D; fromEvent(this.btn.nativeElement, &quot;click&quot;)
 			.subscribe((event: Event) &#x3D;&gt; {
 				// Add/Remove event listener based on isOpen to improve performance when there
 				// are a lot of toggletips
@@ -2256,6 +2288,10 @@ export class Toggletip extends PopoverContainer implements AfterViewInit {
 		}
 	}
 
+	ngOnDestroy(): void {
+		this.subscription.unsubscribe();
+	}
+
 	private handleExpansion(state &#x3D; false, event: Event) {
 		this.handleChange(state, event);
 		if (this.btn) {

package/docs/documentation/coverage.html

@@ -641,9 +641,9 @@
             </td>
             <td>directive</td>
             <td>DialogDirective</td>
-            <td align="right" data-sort="69">
-                <span class="coverage-percent">69 %</span>
-                <span class="coverage-count">(25/36)</span>
+            <td align="right" data-sort="67">
+                <span class="coverage-percent">67 %</span>
+                <span class="coverage-count">(25/37)</span>
             </td>
         </tr>
         <tr class="very-good">
@@ -725,9 +725,9 @@
             </td>
             <td>directive</td>
             <td>OverflowMenuDirective</td>
-            <td align="right" data-sort="61">
-                <span class="coverage-percent">61 %</span>
-                <span class="coverage-count">(26/42)</span>
+            <td align="right" data-sort="60">
+                <span class="coverage-percent">60 %</span>
+                <span class="coverage-count">(26/43)</span>
             </td>
         </tr>
         <tr class="low">
@@ -2897,9 +2897,9 @@
             </td>
             <td>component</td>
             <td>Toggletip</td>
-            <td align="right" data-sort="38">
-                <span class="coverage-percent">38 %</span>
-                <span class="coverage-count">(15/39)</span>
+            <td align="right" data-sort="35">
+                <span class="coverage-percent">35 %</span>
+                <span class="coverage-count">(14/40)</span>
             </td>
         </tr>
         <tr class="medium">