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

Explorer validator sorting #7481

Merged
merged 12 commits into from
Jan 23, 2023
Merged

Explorer validator sorting #7481

merged 12 commits into from
Jan 23, 2023

Conversation

Jibz1
Copy link
Contributor

@Jibz1 Jibz1 commented Jan 18, 2023

Screen.Recording.2023-01-20.at.9.17.20.AM.mov

@vercel
Copy link

vercel bot commented Jan 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
explorer ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 23, 2023 at 0:55AM (UTC)
3 Ignored Deployments
Name Status Preview Comments Updated
explorer-storybook ⬜️ Ignored (Inspect) Jan 23, 2023 at 0:55AM (UTC)
frenemies ⬜️ Ignored (Inspect) Jan 23, 2023 at 0:55AM (UTC)
wallet-adapter ⬜️ Ignored (Inspect) Jan 23, 2023 at 0:55AM (UTC)

@Jibz1 Jibz1 requested a review from mystie711 January 18, 2023 19:57
@Jibz1 Jibz1 marked this pull request as ready for review January 18, 2023 19:58
Copy link
Collaborator

@mystie711 mystie711 left a comment

Choose a reason for hiding this comment

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

@Jibz1 following up on our conversation yesterday,

  • default sort by stake
  • remove unsorted state

apps/explorer/src/ui/TableCard.tsx Outdated Show resolved Hide resolved
apps/explorer/src/ui/TableCard.tsx Outdated Show resolved Hide resolved
apps/explorer/src/ui/TableCard.tsx Outdated Show resolved Hide resolved
apps/explorer/src/ui/TableCard.tsx Outdated Show resolved Hide resolved
apps/explorer/src/ui/TableCard.tsx Show resolved Hide resolved
Copy link
Contributor

@Jordan-Mysten Jordan-Mysten left a comment

Choose a reason for hiding this comment

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

Approving to unblock but some nits

apps/explorer/src/pages/validators/Validators.tsx Outdated Show resolved Hide resolved
{
header: '#',
accessorKey: 'number',
cell: (props: any) => (
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't love the any's here. We can follow-up later with an improved Table component, just want to call it out

apps/explorer/src/pages/validators/Validators.tsx Outdated Show resolved Hide resolved
onClick={
column.columnDef.enableSorting
? column.getToggleSortingHandler()
: void 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Just use undefined?

apps/explorer/src/ui/TableCard.tsx Outdated Show resolved Hide resolved
@Jibz1 Jibz1 merged commit 958802e into main Jan 23, 2023
@Jibz1 Jibz1 deleted the explorer-validator-sorting branch January 23, 2023 21:55
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.

4 participants