fix: use AfterViewInit instead of AfterContentInit

Author: sguiheux

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, OnDestroy
+    TemplateRef, ViewContainerRef, ContentChild, EventEmitter, Output, OnDestroy, AfterViewInit
 } from "@angular/core";
 import { Subscription } from "rxjs";
 import { DropdownService, SuiDropdownMenu } from "../../dropdown/internal";
@@ -16,11 +16,11 @@ 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, OnDestroy {
+export abstract class SuiSelectBase<T, U> implements AfterViewInit, OnDestroy {
     public dropdownService:DropdownService;
     public searchService:SearchService<T, U>;
 
-    @ViewChild(SuiDropdownMenu, { static: true })
+    @ViewChild(SuiDropdownMenu)
     protected _menu:SuiDropdownMenu;
 
     // Keep track of all of the rendered select options. (Rendered by the user using *ngFor).
@@ -42,7 +42,10 @@ export abstract class SuiSelectBase<T, U> implements AfterContentInit, OnDestroy
 
     @HostBinding("class.visible")
     public get isVisible():boolean {
-        return this._menu.isVisible;
+        if (this._menu) {
+            return this._menu?.isVisible;
+        }
+        return false;
     }
 
     @Input()
@@ -245,7 +248,7 @@ export abstract class SuiSelectBase<T, U> implements AfterContentInit, OnDestroy
         this.hasClasses = true;
     }
 
-    public ngAfterContentInit():void {
+    public ngAfterViewInit():void {
         this._menu.service = this.dropdownService;
         // We manually specify the menu items to the menu because the @ContentChildren doesn't pick up our dynamically rendered items.
         this._menu.items = this._renderedOptions;
@@ -286,7 +289,7 @@ export abstract class SuiSelectBase<T, U> implements AfterContentInit, OnDestroy
         // The search delay is set to the transition duration to ensure results
         // aren't rendered as the select closes as that causes a sudden flash.
         if (delayed) {
-            this.searchService.searchDelay = this._menu.menuTransitionDuration;
+            this.searchService.searchDelay = this._menu?.menuTransitionDuration;
             this.searchService.updateQueryDelayed("");
         } else {
             this.searchService.updateQuery("");

src/modules/select/components/multi-select.ts

@@ -1,4 +1,12 @@
-import { Component, HostBinding, ElementRef, EventEmitter, Output, Input, Directive } from "@angular/core";
+import {
+    Component,
+    HostBinding,
+    ElementRef,
+    EventEmitter,
+    Output,
+    Input,
+    Directive
+} from "@angular/core";
 import { ICustomValueAccessorHost, KeyCode, customValueAccessorFactory, CustomValueAccessor } from "../../../misc/util/internal";
 import { SuiLocalizationService } from "../../../behaviors/localization/internal";
 import { SuiSelectBase } from "../classes/select-base";

src/modules/select/components/select.ts

@@ -1,4 +1,13 @@
-import { Component, ViewContainerRef, ViewChild, Output, EventEmitter, ElementRef, Directive, Input } from "@angular/core";
+import {
+    Component,
+    ViewContainerRef,
+    ViewChild,
+    Output,
+    EventEmitter,
+    ElementRef,
+    Directive,
+    Input
+} from "@angular/core";
 import { ICustomValueAccessorHost, customValueAccessorFactory, CustomValueAccessor } from "../../../misc/util/internal";
 import { SuiLocalizationService } from "../../../behaviors/localization/internal";
 import { SuiSelectBase } from "../classes/select-base";

src/modules/select/public.ts

@@ -1,10 +1,4 @@
 export {
     SuiSelectModule,
-    IOptionContext,
-    SuiSelect,
-    SuiSelectOption,
-    SuiSelectSearch,
-    SuiSelectValueAccessor,
-    SuiMultiSelect,
-    SuiMultiSelectValueAccessor
+    IOptionContext
 } from "./internal";