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

[LW-10451] Address book name broken on address input #1158

Merged

Conversation

lucas-barros
Copy link

Checklist


Proposed solution

  • Prevent line break on white spaces
  • Add story to include scenario on snapshot test

Testing

Create an address book entry with a white space such as "My address" and verify that the name is displayed correctly in send flow.

Screenshots

image

@lucas-barros lucas-barros marked this pull request as ready for review May 13, 2024 19:34
@lucas-barros lucas-barros requested a review from a team as a code owner May 13, 2024 19:34
Copy link

github-actions bot commented May 13, 2024

Allure Report

allure-report-publisher generated test report!

smokeTests: ✅ test report for 34fea0d9

passed failed skipped flaky total result
Total 30 0 0 0 30

@lucas-barros lucas-barros force-pushed the fix/lw-10451-address-book-name-broken-on-address-input branch from c63926d to e1f00da Compare May 14, 2024 00:33
@DominikGuzei
Copy link
Member

@lucas-barros In the core storybook i can see that the address book name doesn't look right (font color) on dark theme:
image

@lucas-barros
Copy link
Author

lucas-barros commented May 14, 2024

@lucas-barros In the core storybook i can see that the address book name doesn't look right (font color) on dark theme: image

@DominikGuzei that was due to switching from story with global theme to one with local theme. Fixed here: 75de21c.

@lucas-barros lucas-barros force-pushed the fix/lw-10451-address-book-name-broken-on-address-input branch from c216cb8 to 6ae08fc Compare May 14, 2024 13:08
Copy link
Contributor

@renanvalentin renanvalentin left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@VanessaPC VanessaPC left a comment

Choose a reason for hiding this comment

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

LGTM!

@lucas-barros lucas-barros force-pushed the fix/lw-10451-address-book-name-broken-on-address-input branch from 6ae08fc to 34fea0d Compare May 17, 2024 12:26
Copy link

@pczeglik-iohk pczeglik-iohk enabled auto-merge (squash) May 17, 2024 14:59
@pczeglik-iohk pczeglik-iohk disabled auto-merge May 17, 2024 15:00
@pczeglik-iohk pczeglik-iohk merged commit 90d84e9 into main May 17, 2024
14 of 15 checks passed
@pczeglik-iohk pczeglik-iohk deleted the fix/lw-10451-address-book-name-broken-on-address-input branch May 17, 2024 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants