Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

[FEATURE] Add color to transfer label in transaction list #3637

Closed
3 tasks done
shinebrillant opened this issue Oct 20, 2024 · 8 comments · Fixed by #3647
Closed
3 tasks done

[FEATURE] Add color to transfer label in transaction list #3637

shinebrillant opened this issue Oct 20, 2024 · 8 comments · Fixed by #3647
Assignees
Labels
approved Approved by the Ivy Wallet team. Ready for dev user request Feature/improvement requested by an user

Comments

@shinebrillant
Copy link
Contributor

shinebrillant commented Oct 20, 2024

Please confirm the following:

  • I've checked the current issues for duplicate issues.
  • I've requested a single (only one) feature/change in this issue. It complies with the One Request Per GitHub Issue (ORPGI) rule.
  • My issue is well-defined and describes how it should be implemented from UI/UX perspective.

What do you want to be added or improved?

[Feature] Added color to accounts labels in transactions list Behind IvyFeatureToggle #3628
Accounts and categories have added colors, and transfers are also required.

Why do you need it?

Easy to identify.

How do you imagine it?

No response

556+456nsmail-1

@shinebrillant shinebrillant added the user request Feature/improvement requested by an user label Oct 20, 2024
@ivywallet
Copy link
Collaborator

Thank you @shinebrillant for raising Issue #3637! 🚀
What's next? Read our Contribution Guidelines 📚.

Tagging @ILIYANGERMANOV for review & approval 👀

@ILIYANGERMANOV ILIYANGERMANOV added the approved Approved by the Ivy Wallet team. Ready for dev label Oct 21, 2024
@viratde
Copy link
Contributor

viratde commented Oct 24, 2024

I'm on it

@ivywallet
Copy link
Collaborator

Thank you for your interest @viratde! 🎉
Issue #3637 is assigned to you. You can work on it! ✅

If you don't want to work on it now, please un-assign yourself so other contributors can take it.

Also, make sure to read our Contribution Guidelines.

@viratde
Copy link
Contributor

viratde commented Oct 24, 2024

Hey @ILIYANGERMANOV, then what is this feature request about?

@ILIYANGERMANOV
Copy link
Collaborator

@viratde it's about adding color to the transfers and it should be disabled by default. Most users prefer not having colored accounts. @shinebrillant can confirm if I understand it correctly

@viratde
Copy link
Contributor

viratde commented Oct 24, 2024

Hey @shinebrillant , if you could attach a video or image, that would be really helpful

@shinebrillant
Copy link
Contributor Author

Enable advanced features to show account colors, transfers will not show colors.

@viratde
Copy link
Contributor

viratde commented Oct 25, 2024

Hey @ILIYANGERMANOV, Could you please review the pr attached?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Approved by the Ivy Wallet team. Ready for dev user request Feature/improvement requested by an user
Projects
None yet
4 participants