Better icon for zxtns with no memo #42

Closed
opened 3 years ago by duke · 5 comments
duke commented 3 years ago
Owner

Currently it's a white box that almost looks like a broken image.

Originally: https://github.com/MyHush/SilentDragon/issues/145

Currently it's a white box that almost looks like a broken image. Originally: https://github.com/MyHush/SilentDragon/issues/145
Collaborator

This is more noticeable in different themes. I wasn't sure if by design to keep less busy and easier to see txs with a memo. I can look into adding different icons for sent, received, and taddr->zaddr. Also related to #40

This is more noticeable in different themes. I wasn't sure if by design to keep less busy and easier to see txs with a memo. I can look into adding different icons for sent, received, and taddr->zaddr. Also related to https://git.hush.is/hush/SilentDragon/issues/40
Poster
Owner

@fekt I think different/better icons would be an improvement

@fekt I think different/better icons would be an improvement
Collaborator

I checked this quick and there are currently no icons in resources. It uses a default icon for memo or SD logo if requesting payment. Everything else gets set to an empty pixmap. Relevant code: https://git.hush.is/hush/SilentDragon/src/branch/dev/src/txtablemodel.cpp#L163

I'll grab icons from somewhere and see what I can do later.

I checked this quick and there are currently no icons in resources. It uses a default icon for memo or SD logo if requesting payment. Everything else gets set to an empty pixmap. Relevant code: https://git.hush.is/hush/SilentDragon/src/branch/dev/src/txtablemodel.cpp#L163 I'll grab icons from somewhere and see what I can do later.
Collaborator

Added in this commit: af98833558

Added in this commit: https://git.hush.is/hush/SilentDragon/commit/af98833558c66397dba33baf3b749bede2daa1ca
duke commented 1 year ago
Poster
Owner

calling this done, closing

calling this done, closing
duke closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.