chore(providers): fixed typeahead error

Author: Alex Umanskiy

CHANGELOG.md

@@ -1,5 +1,4 @@
 
-
 ## [21.2.0](https://github.com/valor-software/ngx-bootstrap/compare/v21.0.1...v21.2.0) (2025-04-13)
 
 ### BREAKING CHANGES
@@ -9,6 +8,8 @@
 * **inputs:** All `@Input()` decorators have been replaced with the `input()` signal function API. Programmatic access to component inputs now requires signal read syntax (e.g., `component.myInput()` instead of `component.myInput`).
 * **outputs:** All `@Output()` decorators have been replaced with the `output()` function API. Output emissions continue to use `.emit()`.
 * **services:** Removed `NgZone` dependency from carousel, component-loader, positioning service, and focus-trap. Replaced `NgZone.onStable` with `afterRenderEffect()`.
+* **modules:** Removed deprecated `forRoot()` static method from all 17 module classes (AccordionModule, AlertModule, ButtonsModule, CarouselModule, CollapseModule, BsDatepickerModule, BsDropdownModule, FocusTrapModule, PaginationModule, PopoverModule, ProgressbarModule, RatingModule, SortableModule, TabsModule, TimepickerModule, TooltipModule, TypeaheadModule). These methods were no-ops returning empty providers and were deprecated since v18. Simply use the module directly in imports (e.g., `TooltipModule` instead of `TooltipModule.forRoot()`).
+* **schematics:** `ng add` schematics now generate module imports without `.forRoot()`. 
 
 ### Features
 

apps/ngx-bootstrap-docs/src/ng-api-doc.ts

@@ -1736,81 +1736,82 @@ export const ngdoc: any = {
       },
       {
         name: 'show',
-        description:
-          '<p>Opens an element&#39;s popover. This is considered a &quot;manual&quot; triggering of\nthe popover.</p>\n',
+        description: '<p>Opens an element’s popover. This is considered a “manual” triggering of\nthe popover.</p>\n',
         args: [],
         returnType: 'void'
       },
       {
         name: 'hide',
-        description:
-          '<p>Closes an element&#39;s popover. This is considered a &quot;manual&quot; triggering of\nthe popover.</p>\n',
+        description: '<p>Closes an element’s popover. This is considered a “manual” triggering of\nthe popover.</p>\n',
         args: [],
         returnType: 'void'
       },
       {
         name: 'toggle',
-        description:
-          '<p>Toggles an element&#39;s popover. This is considered a &quot;manual&quot; triggering of\nthe popover.</p>\n',
+        description: '<p>Toggles an element’s popover. This is considered a “manual” triggering of\nthe popover.</p>\n',
         args: [],
         returnType: 'void'
       }
     ],
     properties: [
       {
         name: 'adaptivePosition',
-        type: 'InputSignal<boolean>',
+        defaultValue: 'true',
+        type: 'boolean',
         description: '<p>sets disable adaptive position</p>\n'
       },
       {
         name: 'container',
-        type: 'InputSignal<string>',
+        type: 'string',
         description: '<p>A selector specifying the element the popover should be appended to.</p>\n'
       },
       {
         name: 'containerClass',
-        type: 'InputSignal<string>',
+        type: 'string',
         description: '<p>Css class for popover container</p>\n'
       },
       {
         name: 'delay',
-        type: 'InputSignal<number>',
+        defaultValue: '0',
+        type: 'number',
         description: '<p>Delay before showing the tooltip</p>\n'
       },
       {
         name: 'isOpen',
-        type: 'boolean',
+        type: 'ModelSignal<boolean>',
         description: '<p>Returns whether or not the popover is currently being shown</p>\n'
       },
       {
         name: 'onHidden',
-        type: 'EventEmitter<unknown>',
+        type: 'OutputEmitterRef<unknown>',
         description: '<p>Emits an event when the popover is hidden</p>\n'
       },
       {
         name: 'onShown',
-        type: 'EventEmitter<unknown>',
+        type: 'OutputEmitterRef<unknown>',
         description: '<p>Emits an event when the popover is shown</p>\n'
       },
       {
         name: 'outsideClick',
-        type: 'InputSignal<boolean>',
+        defaultValue: 'false',
+        type: 'boolean',
         description: '<p>Close popover on outside click</p>\n'
       },
       {
         name: 'placement',
-        type: 'InputSignal<AvailableBSPositions>',
+        defaultValue: 'top',
+        type: 'AvailableBSPositions',
         description:
           '<p>Placement of a popover. Accepts: &quot;top&quot;, &quot;bottom&quot;, &quot;left&quot;, &quot;right&quot;</p>\n'
       },
       {
         name: 'popover',
-        type: 'InputSignal<string | TemplateRef<any>>',
+        type: 'string | TemplateRef<any>',
         description: '<p>Content to be displayed as popover.</p>\n'
       },
       {
         name: 'popoverContext',
-        type: 'InputSignal<any>',
+        type: 'any',
         description: '<p>Context to be used if popover is a template.</p>\n'
       },
       {
@@ -1820,12 +1821,13 @@ export const ngdoc: any = {
       },
       {
         name: 'popoverTitle',
-        type: 'InputSignal<string>',
+        type: 'string',
         description: '<p>Title of a popover.</p>\n'
       },
       {
         name: 'triggers',
-        type: 'InputSignal<string>',
+        defaultValue: 'click',
+        type: 'string',
         description: '<p>Specifies events that should trigger. Supports a space separated list of\nevent names.</p>\n'
       }
     ]

src/typeahead/typeahead-container.component.ts

@@ -23,8 +23,6 @@ import { typeaheadAnimation } from './typeahead-animations';
 import { TypeaheadOptionItemContext, TypeaheadOptionListContext, TypeaheadTemplateMethods } from './models';
 import { NgTemplateOutlet } from '@angular/common';
 
-let nextWindowId = 0;
-
 @Component({
     selector: 'typeahead-container',
     templateUrl: './typeahead-container.component.html',
@@ -71,7 +69,7 @@ export class TypeaheadContainerComponent implements OnDestroy {
   animationState?: string;
   positionServiceSubscription = new Subscription();
   height = 0;
-  popupId = `ngb-typeahead-${nextWindowId++}`;
+  popupId = '';
 
   get typeaheadTemplateMethods(): TypeaheadTemplateMethods {
     return {
@@ -96,7 +94,6 @@ export class TypeaheadContainerComponent implements OnDestroy {
     public element: ElementRef,
     private changeDetectorRef: ChangeDetectorRef
   ) {
-    this.renderer.setAttribute(this.element.nativeElement, 'id', this.popupId);
     this.positionServiceSubscription.add(this.positionService.event$?.subscribe(
       () => {
         if (this.isAnimated) {

src/typeahead/typeahead.directive.ts

@@ -26,6 +26,8 @@ import { getValueFromObject, latinize, tokenize } from './typeahead-utils';
 import { TypeaheadConfig } from './typeahead.config';
 import { PositioningService } from 'ngx-bootstrap/positioning';
 
+let nextWindowId = 0;
+
 // eslint-disable-next-line
 type TypeaheadOption = string | Record<string | number, any>;
 type TypeaheadOptionArr = TypeaheadOption[] | Observable<TypeaheadOption>;
@@ -35,7 +37,7 @@ type TypeaheadOptionArr = TypeaheadOption[] | Observable<TypeaheadOption>;
     exportAs: 'bs-typeahead',
     host: {
         '[attr.aria-activedescendant]': 'activeDescendant',
-        '[attr.aria-owns]': 'isOpen ? this._container?.popupId : null',
+        '[attr.aria-owns]': 'isOpen ? popupId : null',
         '[attr.aria-expanded]': 'isOpen',
         '[attr.aria-autocomplete]': 'list'
     },
@@ -171,6 +173,7 @@ export class TypeaheadDirective implements OnInit, OnDestroy {
 
   activeDescendant?: string;
   isOpen = false;
+  popupId = `ngb-typeahead-${nextWindowId++}`;
   list = 'list';
   _container?: TypeaheadContainerComponent;
   isActiveItemChanged = false;
@@ -401,6 +404,8 @@ export class TypeaheadDirective implements OnInit, OnDestroy {
     }
 
     this._container = this._typeahead.instance;
+    this._container.popupId = this.popupId;
+    this.renderer.setAttribute(this._container.element.nativeElement, 'id', this.popupId);
     this._container.parent = this;
     // This improves the speed as it won't have to be done for each list item