subhranshudash13-dotcom commented on issue #587: URL: https://github.com/apache/fineract-backoffice-ui/issues/587#issuecomment-5774872633
Hi @Aman-Mittal, I’d like to work on this issue! Based on the write-up, I’m planning to tackle it in these steps: 1. Check related PRs/issues first — I’ll verify whether #575/#584 have already merged and follow their token-naming conventions so the changes stay consistent across themes. 2. Update the Pending/Active status colors — I’ll add/update the text-color tokens for the status pills while keeping their existing tinted backgrounds, using the suggested ~"#8A5300" / "#126330" values. 3. Add the "--primary-text" token — I’ll introduce this for places where "--primary-color" is currently used as foreground text, including links, active tab labels, section headings, and the Mark All Read fill. I’ll follow the existing "--primary-strong" pattern used for the Guide button. 4. Update the affected theme files — I’ll apply the new tokens wherever needed and make sure the naming/usage stays consistent across the themes. 5. Verify the contrast changes — I’ll compare the before/after contrast ratios for the affected elements and include those results in my update. 6. Keep the link-color unification separate — I’ll leave the "#3F51B5" vs "#3498DB" difference as a follow-up so this PR stays focused, unless you’d prefer me to address it here as well. Could you assign this to me? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
