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

Various PIVX Promos fixes #498

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Various PIVX Promos fixes #498

wants to merge 2 commits into from

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Dec 12, 2024

Abstract

This PR fixes various PIVX Promos UX issues.

  • Re-adds the CSV Export button, resolves Bring back CSV Exports for PIVX Promos #491
  • Move the PIV ticker outside of in-line in replacement of the Amount title, for extra table space.
  • Change the "default-max code length" to a const and use it everywhere.
  • Remove additional entropy on codes over MAX_LENGTH, which is currently 10.
  • Use the unicode ellipsis instead of a manual ... one, which is smaller, as it's a single character length.

Example of a >10 length code and an exactly 10 length code, rendering nicely without any wrapping.
image


Testing

To test this PR, it's suggested to attempt these user flows, or variations of these:

  • Test CSV exports for PIVX Promos.
  • Test that <10 length codes are created with entropy, 10 length codes have no entropy or cutoff, and >10 have no entropy but cut-off to prevent overflow scrolling.

If any errors are found, the PR works unexpectedly, or you have viable suggestions to improve the UX or functionality of the PR, let me know!


@JSKitty JSKitty added the Bug This is either a bugfix (PR) or a bug (issue). label Dec 12, 2024
@JSKitty JSKitty self-assigned this Dec 12, 2024
Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for cheery-moxie-4f1121 ready!

Name Link
🔨 Latest commit bb12f1f
🔍 Latest deploy log https://app.netlify.com/sites/cheery-moxie-4f1121/deploys/675c6ba17f655500088b08da
😎 Deploy Preview https://deploy-preview-498--cheery-moxie-4f1121.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is either a bugfix (PR) or a bug (issue).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bring back CSV Exports for PIVX Promos
1 participant