Skip to content

feat: add blueprint list#385

Merged
pedroferreira1 merged 10 commits intomasterfrom
feat/blueprint-list
Feb 26, 2025
Merged

feat: add blueprint list#385
pedroferreira1 merged 10 commits intomasterfrom
feat/blueprint-list

Conversation

@pedroferreira1
Copy link
Copy Markdown
Member

@pedroferreira1 pedroferreira1 commented Feb 17, 2025

Acceptance Criteria

  • Add new Navigation for nano
  • Add blueprint list with two tabs (on chain and built in)
  • Refactor EllipsisCell component to receive the elements count as props because now we have different counts depending on the component
  • Fix missing arrow in sorting header for Tokens table
blueprint.list.1.mov
blueprint.list.2.mov

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

Comment on lines -153 to +158
return 'page-item active';
return 'page-item';
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ana decided to remove the active style from table pagination buttons, so both pagination buttons have the same style now.

tuliomir
tuliomir previously approved these changes Feb 19, 2025
Comment on lines +5 to +7
// XXX We should use function component with SortableTable as a component
// but renderTableHead and renderTableBody are implemented and not
// expected as a props, so it demands a bigger refactor
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⭐ Thanks for highlighting this tech debt here! This is pending since before the UX revamp.

tuliomir
tuliomir previously approved these changes Feb 22, 2025
@pedroferreira1 pedroferreira1 moved this from In Review (Done) to In Review (WIP) in Hathor Network Feb 25, 2025
@tuliomir tuliomir moved this from In Review (WIP) to In Review (Done) in Hathor Network Feb 26, 2025
@pedroferreira1 pedroferreira1 merged commit eeea785 into master Feb 26, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Feb 26, 2025
@pedroferreira1 pedroferreira1 deleted the feat/blueprint-list branch February 26, 2025 14:47
@pedroferreira1 pedroferreira1 moved this from Waiting to be deployed to Done in Hathor Network Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants