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

Tx Description Box #294

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

Tx Description Box #294

wants to merge 3 commits into from

Conversation

Duddino
Copy link
Member

@Duddino Duddino commented Feb 13, 2024

Abstract

Adds a tx description box when clicking on a tx in the activity.
TODO: Improve the UI

Copy link

netlify bot commented Feb 13, 2024

Deploy Preview for cheery-moxie-4f1121 ready!

Name Link
🔨 Latest commit 66e12e4
🔍 Latest deploy log https://app.netlify.com/sites/cheery-moxie-4f1121/deploys/65ddf7ae7a4536000808998f
😎 Deploy Preview https://deploy-preview-294--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.

@Duddino Duddino self-assigned this Feb 13, 2024
@Duddino Duddino added the Enhancement New feature or request label Feb 13, 2024
@JSKitty
Copy link
Member

JSKitty commented Feb 13, 2024

Nice idea, just tested quickly - we could expand this functionally quite a bit, IMO, moving the 'Explorer URL' hyperlink in to this dialog, then making the entire 'TX History Element' open this dialog, which can display the in/outs, TX fee, explorer URL, and potentially any other info related to the TX.

Me and Bread + Meerkat can work on the design when needed, assuming unfinished with the Draft enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants