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(DashboardTableEfficiency): update UI for relative values in th… #1149

Draft
wants to merge 3 commits into
base: staging
Choose a base branch
from

Conversation

benji-bitfly
Copy link
Contributor

…e validator summary

Display - when relative values are equal to 0.

See: BEDS-844

Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2024

Deploying beaconchain with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7b6d885
Status: ✅  Deploy successful!
Preview URL: https://10aaa5e9.beaconchain.pages.dev
Branch Preview URL: https://beds-844-relative-values-in.beaconchain.pages.dev

View logs

@benji-bitfly benji-bitfly marked this pull request as ready for review November 25, 2024 09:43
@benji-bitfly benji-bitfly force-pushed the BEDS-844/relative-values-in-summary branch from afac4dd to 52b6fcc Compare November 25, 2024 15:27
@benji-bitfly benji-bitfly marked this pull request as draft November 26, 2024 08:30
@benji-bitfly benji-bitfly force-pushed the BEDS-844/relative-values-in-summary branch 8 times, most recently from 8827ea9 to d62d1df Compare November 27, 2024 15:34
…equal to `0`

Improve `UX` by displaying `-` for `0` values.

See: BEDS-844
It is easier to work with components if their name reflect the file path in nuxt.
@benji-bitfly benji-bitfly force-pushed the BEDS-844/relative-values-in-summary branch from d62d1df to 7b6d885 Compare November 27, 2024 15:36
@marcel-bitfly
Copy link
Contributor

np: commit-msgs

feat(DashboardTableEfficiency): display `-` when relative values are equal to `0`

See: BEDS-844

<BcFormatNumber
:class="data.failedClass"
:value="props.failed"
<span v-if="props.success === 0 && props.failed === 0">0 / 0</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

how would 50/70 be displayed?

  1. 50/50 or
  2. 50 / 50

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