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

chore: adjust asset item #121

Merged
merged 1 commit into from
Nov 16, 2023
Merged

chore: adjust asset item #121

merged 1 commit into from
Nov 16, 2023

Conversation

pedronauck
Copy link
Contributor

⭐ Closes FE-17

@pedronauck pedronauck self-assigned this Nov 16, 2023
Copy link

linear bot commented Nov 16, 2023

FE-17 Change Asset Component Name

Currently, the asset component displays the name, symbol, and asset_id. But we can shorten it by removing the asset_id from the side of the name and adding a hover that displays it. If the asset is not a known asset, we should just display the asset_id instead of the name and symbol. We can make the design more similar to what we have here: https://www.figma.com/file/VTWcuPxsUtKZDk0iKn3y7T/Fuel-Portal-Product-Board?type=design&node-id=2068-72107&mode=design&t=xxQXAPs6q4kaYnUk-4

Copy link
Contributor

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 0/0
🟢 Branches 100% 0/0
🟢 Functions 100% 0/0
🟢 Lines 100% 0/0

Test suite run success

0 tests passing in 0 suite.

Report generated by 🧪jest coverage report action from c49c776

@pedronauck pedronauck merged commit ed78ba0 into main Nov 16, 2023
7 checks passed
@pedronauck pedronauck deleted the pn/fe-17/asset-component branch November 16, 2023 20:23
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.

2 participants