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.

Sign up for GitHub

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

Jump to bottom

Replace use-theme-value with use-theme #19602

Closed
clauxx opened this issue Apr 10, 2024 · 0 comments
Closed

Replace use-theme-value with use-theme #19602

clauxx opened this issue Apr 10, 2024 · 0 comments
Assignees
@flexsurfer
Labels
developer-xp tech-debt

Comments

@clauxx
Copy link
Member

clauxx commented Apr 10, 2024

Bug Report

Problem

follow-up to this discussion thread

Make use-theme return the theme value instead of a map and replace its usages
@clauxx clauxx mentioned this issue Apr 10, 2024
Merged
@flexsurfer flexsurfer self-assigned this Apr 16, 2024
flexsurfer added a commit that referenced this issue Apr 16, 2024
@flexsurfer
Replace use-theme-value with use-theme
13661ec 
#19602
@flexsurfer flexsurfer mentioned this issue Apr 16, 2024
Closed
flexsurfer added a commit that referenced this issue Apr 18, 2024
@flexsurfer
Replace use-theme-value with use-theme
d46fcc5 
#19602
flexsurfer added a commit that referenced this issue Apr 18, 2024
@flexsurfer
Replace use-theme-value with use-theme #19602
5dfc502
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@flexsurfer flexsurfer

Labels
developer-xp tech-debt
Projects
👨‍🎤🪄 Voguish Magicians [mobile] (Usability and readiness for public beta)
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace use-theme-value with use-theme #19602 status-im/status-mobile
3 participants
@flexsurfer @Parveshdhull @clauxx