Skip to content

Commit

Permalink
fix: align controller-mixin API visibility (#6951)
Browse files Browse the repository at this point in the history
  • Loading branch information
tomivirkki authored Dec 11, 2023
1 parent 713c37a commit f3885f9
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions packages/component-base/src/controller-mixin.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
* This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
*/
import type { Constructor } from '@open-wc/dedupe-mixin';
import type { ReactiveController, ReactiveControllerHost } from 'lit';
import type { ReactiveController } from 'lit';

/**
* A mixin for connecting controllers to the element.
Expand All @@ -13,16 +13,14 @@ export declare function ControllerMixin<T extends Constructor<HTMLElement>>(
superclass: T,
): Constructor<ControllerMixinClass> & T;

export declare class ControllerMixinClass
implements Pick<ReactiveControllerHost, 'addController' | 'removeController'>
{
export declare class ControllerMixinClass {
/**
* Registers a controller to participate in the element update cycle.
*/
addController(controller: ReactiveController): void;
protected addController(controller: ReactiveController): void;

/**
* Removes a controller from the element.
*/
removeController(controller: ReactiveController): void;
protected removeController(controller: ReactiveController): void;
}

0 comments on commit f3885f9

Please sign in to comment.