Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empty state component #476

Merged
merged 8 commits into from
Feb 21, 2025
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions addon/components/o-s-s/empty-state.hbs
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
<div class="fx-1 fx-col fx-xalign-center fx-gap-px-18" ...attributes>
{{#if (has-block "image")}}
{{yield to="image"}}
{{else if @badgeIcon}}
<OSS::Badge @icon={{@badgeIcon}} @size="lg" />
{{/if}}
<span class="fx-col fx-xalign-center fx-gap-px-6">
<div class="font-color-gray-900 font-size-{{this.titleSize}} font-weight-semibold">
{{@title}}
</div>
<div class="font-color-gray-500 font-size-{{this.size}}">
{{@subtitle}}
</div>
</span>
{{#if (has-block "actions")}}
{{yield to="actions"}}
{{/if}}
</div>
95 changes: 95 additions & 0 deletions addon/components/o-s-s/empty-state.stories.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,95 @@
import { hbs } from 'ember-cli-htmlbars';

export default {
title: 'Components/OSS::EmptyState',
component: 'empty state',
argTypes: {
badgeIcon: {
description: 'a font-awesome icon to be displayed in a badge',
table: {
type: {
summary: 'string'
},
defaultValue: { summary: 'undefined' }
},
control: { type: 'text' }
},
title: {
description: 'A title displayed below the icon or badge in the component',
table: {
type: {
summary: 'string'
},
defaultValue: { summary: 'undefined' }
},
control: { type: 'text' }
},
subtitle: {
description: 'A subtitle displayed under the title in the component',
table: {
type: {
summary: 'string'
},
defaultValue: { summary: 'undefined' }
},
control: { type: 'text' }
},
size: {
description: 'The size of the state',
table: {
type: {
summary: 'string'
},
defaultValue: { summary: 'undefined' }
},
control: { type: 'select' },
options: ['sm', 'md']
}
},
parameters: {
docs: {
description: {
component: 'An component used when there is nothing to display on a page'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small typo here, should be A component instead of an :)

}
}
}
};

const defaultArgs = {
badgeIcon: 'fa-thumbs-up',
title: 'Empty State Title',
subtitle: 'Additional information here',
size: 'md'
};

const Template = (args) => ({
template: hbs`<OSS::EmptyState @badgeIcon={{this.badgeIcon}} @title={{this.title}} @subtitle={{this.subtitle}} @size={{this.size}} />`,
context: args
});

const ImageTemplate = (args) => ({
template: hbs`<OSS::EmptyState @badgeIcon={{this.badgeIcon}} @title={{this.title}} @subtitle={{this.subtitle}} @size={{this.size}}>
<:image>
<OSS::Illustration @src="/@upfluence/oss-components/assets/images/no-records.svg" />
</:image>
</OSS::EmptyState>`,
context: args
});

const ActionTemplate = (args) => ({
template: hbs`<OSS::EmptyState @badgeIcon={{this.badgeIcon}} @title={{this.title}} @subtitle={{this.subtitle}} @size={{this.size}}>
<:actions>
<OSS::Button @label="Click me" />
</:actions>
</OSS::EmptyState>`,
context: args
});

export const Default = Template.bind({});
Default.args = defaultArgs;

export const UsageWithImage = ImageTemplate.bind({});
UsageWithImage.args = defaultArgs;

export const UsageWithActions = ActionTemplate.bind({});
UsageWithActions.args = defaultArgs;
21 changes: 21 additions & 0 deletions addon/components/o-s-s/empty-state.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
import type { htmlSafe } from '@ember/template';
import Component from '@glimmer/component';

interface OSSEmptyStateComponentSignature {
badgeIcon?: string;
title: ReturnType<typeof htmlSafe>;
subtitle: ReturnType<typeof htmlSafe>;
size?: 'sm' | 'md';
}

const ALLOWED_SIZE = ['sm', 'md'];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const ALLOWED_SIZE = ['sm', 'md'];
const ALLOWED_SIZES = ['sm', 'md'];

My bad I forget the "s" at the end, don't forget the type also


export default class OSSEmptyStateComponent extends Component<OSSEmptyStateComponentSignature> {
get titleSize(): string {
return this.size === 'sm' ? 'md' : 'lg';
}

get size(): string {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: maybe have subtitleSize here to match naming scheme with titleSize ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have mixed feelings about this because it's also used for invalid values handling
Or should I add a subtitleSize getter that just pass the size getter ?

return this.args.size && ALLOWED_SIZE.includes(this.args.size) ? this.args.size : 'md';
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you could add a size getter that returns this.args.size ?? 'md' and use it locally. or even:

get size(): string {
  return ['sm', 'md'].includes(this.args.size) ? this.args.size : 'md';
}

(so we properly handle invalid sizes)

1 change: 1 addition & 0 deletions app/components/o-s-s/empty-state.js
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export { default } from '@upfluence/oss-components/components/o-s-s/empty-state';
40 changes: 40 additions & 0 deletions tests/dummy/app/templates/index.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -276,4 +276,44 @@
</div>
</div>

<div
class="fx-col fx-1 background-color-white border border-color-default border-radius-md padding-px-12 fx-gap-px-12"
>
<div class="font-size-md font-weight-semibold">
Empty state
</div>
<div class="fx-row fx-gap-px-36 fx-xalign-start">
<OSS::EmptyState @title="Title" @subtitle="Subtitle">
<:image>
<OSS::Illustration @src="/@upfluence/oss-components/assets/images/no-records.svg" />
</:image>
<:actions>
<OSS::Button @label="Button" />
<OSS::Button @skin="primary" @label="Button" />
</:actions>
</OSS::EmptyState>
<OSS::EmptyState @title="Title" @subtitle="Subtitle" @badgeIcon="fa-thumbs-up">
<:actions>
<OSS::Button @label="Button" />
<OSS::Button @skin="primary" @label="Button" />
</:actions>
</OSS::EmptyState>
<OSS::EmptyState @title="Title" @subtitle="Subtitle" @size="sm">
<:image>
<OSS::Illustration @src="/@upfluence/oss-components/assets/images/no-records.svg" />
</:image>
<:actions>
<OSS::Button @label="Button" />
<OSS::Button @skin="primary" @label="Button" />
</:actions>
</OSS::EmptyState>
<OSS::EmptyState @title="Title" @subtitle="Subtitle" @size="sm" @badgeIcon="fa-thumbs-up">
<:actions>
<OSS::Button @label="Button" />
<OSS::Button @skin="primary" @label="Button" />
</:actions>
</OSS::EmptyState>
</div>
</div>

</div>
77 changes: 77 additions & 0 deletions tests/integration/components/o-s-s/empty-state-test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,77 @@
import { module, test } from 'qunit';
import { setupRenderingTest } from 'ember-qunit';
import { render } from '@ember/test-helpers';
import { hbs } from 'ember-cli-htmlbars';

module('Integration | Component | o-s-s/empty-state', function (hooks) {
setupRenderingTest(hooks);

test('it renders with default properties', async function (assert) {
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" />`);

assert.dom('div.font-color-gray-900').hasText('No Data');
assert.dom('div.font-color-gray-500').hasText('Try again later');
});

test('it renders with a badge icon', async function (assert) {
this.set('image', 'fa-thumbs-up');
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" @badgeIcon={{this.image}} />`);

assert.dom('div.font-color-gray-900').hasText('No Data');
assert.dom('div.font-color-gray-500').hasText('Try again later');
assert.dom('.upf-badge').exists();
});

test('it supports named-block usage for image', async function (assert) {
await render(hbs`
<OSS::EmptyState @title="No Data" @subtitle="Try again later">
<:image>
<img src="/test-image.png" alt="Test Image" />
</:image>
</OSS::EmptyState>
`);

assert.dom('div.font-color-gray-900').hasText('No Data');
assert.dom('div.font-color-gray-500').hasText('Try again later');
assert.dom('img').exists();
});

test('it supports named-block usage for actions', async function (assert) {
await render(hbs`
<OSS::EmptyState @title="No Data" @subtitle="Try again later">
<:actions>
<button type="button">Retry</button>
</:actions>
</OSS::EmptyState>
`);

assert.dom('div.font-color-gray-900').hasText('No Data');
assert.dom('div.font-color-gray-500').hasText('Try again later');
assert.dom('button').hasText('Retry');
});

module('component size', function (hooks) {
test('it applies md sizes by default', async function (assert) {
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" />`);
assert.dom('div.font-color-gray-900').hasClass('font-size-lg');
assert.dom('div.font-color-gray-500').hasClass('font-size-md');
});
test('it applies md sizes when given a wrong size', async function (assert) {
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" @size="wrong" />`);
assert.dom('div.font-color-gray-900').hasClass('font-size-lg');
assert.dom('div.font-color-gray-500').hasClass('font-size-md');
});

test('it applies md sizes when specified', async function (assert) {
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" @size="md" />`);
assert.dom('div.font-color-gray-900').hasClass('font-size-lg');
assert.dom('div.font-color-gray-500').hasClass('font-size-md');
});

test('it applies sm sizes when specified', async function (assert) {
await render(hbs`<OSS::EmptyState @title="No Data" @subtitle="Try again later" @size="sm" />`);
assert.dom('div.font-color-gray-900').hasClass('font-size-md');
assert.dom('div.font-color-gray-500').hasClass('font-size-sm');
});
});
});
Loading