Skip to content

Commit

Permalink
refactor: remove unused class members (#10294)
Browse files Browse the repository at this point in the history
**Related Issue:** #10234

## Summary

✨🧹✨
  • Loading branch information
jcfranco authored Sep 13, 2024
1 parent e108f88 commit 2fcf11e
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 52 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -185,8 +185,6 @@ export class ActionBar

resizeObserver = createObserver("resize", (entries) => this.resizeHandlerEntries(entries));

expandToggleEl: HTMLCalciteActionElement;

@State() effectiveLocale: string;

@State() hasActionsEnd = false;
Expand Down Expand Up @@ -344,10 +342,6 @@ export class ActionBar
this.calciteActionBarToggle.emit();
};

setExpandToggleRef = (el: HTMLCalciteActionElement): void => {
this.expandToggleEl = el;
};

updateGroups(): void {
this.setGroupLayout(Array.from(this.el.querySelectorAll("calcite-action-group")));
}
Expand Down Expand Up @@ -409,7 +403,6 @@ export class ActionBar
expandText={messages.expand}
expanded={expanded}
position={position}
ref={this.setExpandToggleRef}
scale={scale}
toggle={toggleExpand}
tooltip={this.expandTooltip}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,6 @@ export class ActionPad
this.setGroupLayout(Array.from(this.el.querySelectorAll("calcite-action-group"))),
);

expandToggleEl: HTMLCalciteActionElement;

@State() effectiveLocale = "";

@Watch("effectiveLocale")
Expand Down Expand Up @@ -231,10 +229,6 @@ export class ActionPad
this.calciteActionPadToggle.emit();
};

setExpandToggleRef = (el: HTMLCalciteActionElement): void => {
this.expandToggleEl = el;
};

updateGroups(): void {
this.setGroupLayout(Array.from(this.el.querySelectorAll("calcite-action-group")));
}
Expand Down Expand Up @@ -288,7 +282,6 @@ export class ActionPad
expandText={messages.expand}
expanded={expanded}
position={position}
ref={this.setExpandToggleRef}
scale={scale}
toggle={toggleExpand}
tooltip={this.expandTooltip}
Expand Down
10 changes: 0 additions & 10 deletions packages/calcite-components/src/components/combobox/combobox.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -810,16 +810,6 @@ export class Combobox
}
};

private toggleCloseEnd = (): void => {
this.open = false;
this.el.removeEventListener("calciteComboboxClose", this.toggleCloseEnd);
};

private toggleOpenEnd = (): void => {
this.open = false;
this.el.removeEventListener("calciteComboboxOpen", this.toggleOpenEnd);
};

onBeforeOpen(): void {
this.scrollToActiveItem();
this.calciteComboboxBeforeOpen.emit();
Expand Down
2 changes: 0 additions & 2 deletions packages/calcite-components/src/components/modal/modal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -356,8 +356,6 @@ export class Modal

modalContent: HTMLDivElement;

initialOverflowCSS: string;

private mutationObserver: MutationObserver = createObserver("mutation", () =>
this.handleMutationObserver(),
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,6 @@ export class TabTitle implements InteractiveComponent, LocalizedComponent, T9nCo
connectLocalized(this);
connectMessages(this);
this.setupTextContentObserver();
this.parentTabNavEl = this.el.closest("calcite-tab-nav");
this.parentTabsEl = this.el.closest("calcite-tabs");
}

Expand Down Expand Up @@ -489,10 +488,6 @@ export class TabTitle implements InteractiveComponent, LocalizedComponent, T9nCo

closeButtonEl: HTMLButtonElement;

containerEl: HTMLDivElement;

parentTabNavEl: HTMLCalciteTabNavElement;

parentTabsEl: HTMLCalciteTabsElement;

resizeObserver = createObserver("resize", () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -199,21 +199,11 @@ export class TimePicker
//
//--------------------------------------------------------------------------

/**
* @internal
*/
@Event({ cancelable: false }) calciteInternalTimePickerBlur: EventEmitter<void>;

/**
* @internal
*/
@Event({ cancelable: false }) calciteInternalTimePickerChange: EventEmitter<string>;

/**
* @internal
*/
@Event({ cancelable: false }) calciteInternalTimePickerFocus: EventEmitter<void>;

//--------------------------------------------------------------------------
//
// Event Listeners
Expand All @@ -224,12 +214,6 @@ export class TimePicker
blurHandler(): void {
this.activeEl = undefined;
this.pointerActivated = false;
this.calciteInternalTimePickerBlur.emit();
}

@Listen("focus")
hostFocusHandler(): void {
this.calciteInternalTimePickerFocus.emit();
}

@Listen("keydown")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,6 @@ export class TreeItem implements ConditionalSlotComponent, InteractiveComponent
[CSS_UTILITY.rtl]: rtl,
}}
data-selection-mode={this.selectionMode}
ref={(el) => (this.defaultSlotWrapper = el as HTMLElement)}
>
{chevron}
{itemIndicator}
Expand Down Expand Up @@ -451,10 +450,6 @@ export class TreeItem implements ConditionalSlotComponent, InteractiveComponent

actionSlotWrapper!: HTMLElement;

childrenSlotWrapper!: HTMLElement;

defaultSlotWrapper!: HTMLElement;

private parentTreeItem?: HTMLCalciteTreeItemElement;

private userChangedValue = false;
Expand Down

0 comments on commit 2fcf11e

Please sign in to comment.