Skip to content

Commit

Permalink
41: Refactor to move properties near where they are relevant.
Browse files Browse the repository at this point in the history
  • Loading branch information
damontgomery committed Jul 16, 2021
1 parent 646a156 commit 2885dd9
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions src/components/base/outline-alert/outline-alert.ts
Expand Up @@ -25,18 +25,6 @@ export const alertStatusTypes: AlertStatusType[] = [
export class OutlineAlert extends OutlineElement {
static styles: CSSResultGroup = [componentStyles];

@property({ type: String })
size: AlertSize = 'large';

@property({ type: String })
statusType: AlertStatusType = 'information';

@property({ type: Boolean })
shouldShowIcon = true;

@property({ type: Boolean })
isInteractive = false;

render(): TemplateResult {
// The `body` wrapper is used to avoid styles (like border) that are preventing us from styling `:host`.
return html`
Expand All @@ -49,6 +37,9 @@ export class OutlineAlert extends OutlineElement {
`;
}

@property({ type: Boolean })
isInteractive = false;

private _iconTemplate(): TemplateResult {
let template = html``;

Expand All @@ -63,6 +54,9 @@ export class OutlineAlert extends OutlineElement {
return template;
}

@property({ type: Boolean })
shouldShowIcon = true;

private _headerTemplate(): TemplateResult {
let template = html``;

Expand All @@ -79,4 +73,10 @@ export class OutlineAlert extends OutlineElement {

return template;
}

@property({ type: String })
size: AlertSize = 'large';

@property({ type: String })
statusType: AlertStatusType = 'information';
}

0 comments on commit 2885dd9

Please sign in to comment.