-
Notifications
You must be signed in to change notification settings - Fork 1
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
Changes from 5 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
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> |
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' | ||
} | ||
} | ||
} | ||
}; | ||
|
||
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 | ||
}); | ||
Miexil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
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; |
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>; | ||||||
Miexil marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
size?: 'sm' | 'md'; | ||||||
} | ||||||
|
||||||
const ALLOWED_SIZE = ['sm', 'md']; | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
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 { | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: maybe have There was a problem hiding this comment. Choose a reason for hiding this commentThe 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 |
||||||
return this.args.size && ALLOWED_SIZE.includes(this.args.size) ? this.args.size : 'md'; | ||||||
} | ||||||
} | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you could add a
(so we properly handle invalid sizes) |
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'; |
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'); | ||
}); | ||
}); | ||
}); |
There was a problem hiding this comment.
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 ofan
:)