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

GC-2490 create FAB component #451

Merged
merged 2 commits into from
Mar 6, 2024
Merged

GC-2490 create FAB component #451

merged 2 commits into from
Mar 6, 2024

Conversation

mike-r-morgan
Copy link
Contributor

JIRA

Technical Design Doc

Description

Screenshots

image

Reviewer Checklist

This section is to be filled out by reviewers

Testing

  • This code was tested by somebody other than the developer. Do not merge until this has been done.

@mike-r-morgan mike-r-morgan requested a review from a team as a code owner March 5, 2024 21:39
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-451.d11k469e311m4w.amplifyapp.com

Copy link
Contributor

@NateWaldschmidt NateWaldschmidt left a comment

Choose a reason for hiding this comment

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

Looks good! One question regarding the Storybook:
image

Should the text wrap, floating card expand, or have text ellipsis? I see it currently wraps but wanted to throw out those thoughts

);

const emits = defineEmits<{
(e: 'button-clicked'): void; // eslint-disable-line no-unused-vars
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: To better match normal emits

Suggested change
(e: 'button-clicked'): void; // eslint-disable-line no-unused-vars
(e: 'click'): void; // eslint-disable-line no-unused-vars

@mike-r-morgan mike-r-morgan merged commit 54de3cd into main Mar 6, 2024
5 checks passed
@mike-r-morgan mike-r-morgan deleted the GC-2490 branch March 6, 2024 17:52
@mike-r-morgan mike-r-morgan mentioned this pull request Mar 6, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants