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

feat(CollectiblesLayout): Always visualize section header if community minted collectibles are present #13948

Merged
merged 4 commits into from
Mar 13, 2024

Conversation

noeliaSD
Copy link
Contributor

Closes #13315

What does the PR do

It closes previous mentioned issue but also some other fixes:

  • Some strange layout behaviour in CollectiblesView caused by a wrong syntax.
  • Storybook waring cleanup.
  • Collectible community tag elided correctly.
  • Visualize section header when only community collectibles present.

Affected areas

Main wallet collectibles layout

Screenshot of functionality

  • Fix in layout due to some syntax error:

Before:
image

After:
Screenshot 2024-03-13 at 11 24 44

  • Community tag:
    Screenshot 2024-03-13 at 11 25 06

  • Header visible/hidden depending on collectibles type:

Screen.Recording.2024-03-13.at.11.24.17.mov

It was missing a correct type in a balance item so it was causing a waring.
Specified text width to correctly elide on the right.
…minted collectibles are present

- If only minted collectibles, section header visible. Otherwise, if only regular collectibles, section hidden.
- Updated header used to `FoldableHeader`.

Closes #13315
@noeliaSD noeliaSD added bug Something isn't working feature labels Mar 13, 2024
@noeliaSD noeliaSD self-assigned this Mar 13, 2024
Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@status-im-auto
Copy link
Member

status-im-auto commented Mar 13, 2024

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 6e1e795 #1 2024-03-13 10:55:32 ~6 min tests/nim 📄log
✔️ 6e1e795 #1 2024-03-13 10:56:21 ~7 min macos/aarch64 🍎dmg
✔️ 6e1e795 #1 2024-03-13 10:59:45 ~10 min macos/x86_64 🍎dmg
✔️ 6e1e795 #1 2024-03-13 11:00:26 ~11 min tests/ui 📄log
✔️ 6e1e795 #1 2024-03-13 11:04:16 ~15 min linux/x86_64 📦tgz
✔️ 6e1e795 #1 2024-03-13 11:26:03 ~36 min windows/x86_64 💿exe

Copy link
Contributor

@dlipicar dlipicar left a comment

Choose a reason for hiding this comment

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

thanks for including those other fixes!

@noeliaSD noeliaSD merged commit f31d613 into master Mar 13, 2024
8 checks passed
@noeliaSD noeliaSD deleted the feat/issue-13315 branch March 13, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Tokens management] Always visualize section header if community minted collectibles are present
4 participants