Skip to content

Smart::Blob - introduce new xs size#689

Merged
Miexil merged 1 commit into
masterfrom
mb/blob-xs-size
Jul 3, 2026
Merged

Smart::Blob - introduce new xs size#689
Miexil merged 1 commit into
masterfrom
mb/blob-xs-size

Conversation

@Miexil

@Miexil Miexil commented Jul 3, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Smart::Blob - introduce new xs size

Screenshot 2026-07-03 at 17 48 53

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation with Figma design link. Don't forget to replace "design" by "file" in the URL. For example https://www.figma.com/file/example
  • Migrated touched components to Glimmer Components
  • Properly labeled

type: 'select'
},
options: ['sm', 'md', 'xl']
options: ['xs', 'sm', 'md', 'xl']

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nitnit: could be DRYer

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@edouardmisset if you mean importing the array from the implem, Storybook can't :p (Ember hell)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

yeah I know.
Actually I meant between L. 20 and L. 26. To have a single source of truth (at least in this file).
But it might not be worth it honestly

@Miexil Miexil merged commit a483590 into master Jul 3, 2026
5 checks passed
@Miexil Miexil deleted the mb/blob-xs-size branch July 3, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants