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

fix: Long Recipient Name in TxRecipientCard #1756

Closed
wants to merge 2 commits into from

Conversation

Dhanraj30
Copy link
Contributor

This fix addresses an issue where long recipient names in the TxRecipientCard component would overflow or break the layout. The solution ensures that recipient names are properly truncated, ellipsed, or wrapped to fit within the designated space, improving the UI's appearance and functionality.

@fuel-cla-bot
Copy link

fuel-cla-bot bot commented Jan 6, 2025

Thanks for the contribution! Before we can merge this, we need @Dhanraj30 to sign the Fuel Labs Contributor License Agreement.

Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

hi @Dhanraj30 thanks for your contribution!

can you do 2 things?
1- put a print on PR description of how the screen is looking like
2- revert the changes from your auto formatter, like changing from ' to ". We wanna keep only changes related to code logic and not formatting

@helciofranco
Copy link
Member

missing to add a changeset also, you can run pnpm changeset

@Dhanraj30 Dhanraj30 requested a review from LuizAsFight January 7, 2025 10:07
@helciofranco
Copy link
Member

should we close this one then @Dhanraj30?

@Dhanraj30
Copy link
Contributor Author

yes

LuizAsFight added a commit that referenced this pull request Jan 20, 2025
Fix Long Recipent name and undo formating 
ref : #1756 , #1752
before changes: 
![Screenshot
(170)](https://github.com/user-attachments/assets/6950ca8c-1dbc-49dc-a6d6-d9151ee74eb5)

after chnages the output
![Screenshot 2025-01-07
143708](https://github.com/user-attachments/assets/8730e718-5d47-43da-b7e7-e9f320237883)

---------

Co-authored-by: Luiz Gomes <[email protected]>
Co-authored-by: Hélcio Franco <[email protected]>
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.

3 participants