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

feat: Store Theme in localStorage. #388

Merged
merged 1 commit into from
Sep 30, 2020
Merged

feat: Store Theme in localStorage. #388

merged 1 commit into from
Sep 30, 2020

Conversation

Jorropo
Copy link
Contributor

@Jorropo Jorropo commented Sep 30, 2020

I've spent 1h30 implementing the same for filters without realizing yolo already somehow remember your builds (I have no idea where it's done but on the https://yolo.berty.io/ build filters are remembered cross reload).

@Jorropo Jorropo requested a review from ekelen as a code owner September 30, 2020 11:02
@auto-add-label auto-add-label bot added the enhancement New feature or request label Sep 30, 2020
@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Sep 30, 2020
Copy link
Collaborator

@ekelen ekelen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Sep 30, 2020
@ekelen
Copy link
Collaborator

ekelen commented Sep 30, 2020

I've spent 1h30 implementing the same for filters without realizing yolo already somehow remember your builds (I have no idea where it's done but on the https://yolo.berty.io/ build filters are remembered cross reload).

They come from the URL params :) Simple but perhaps there is a downside I am overlooking; feel free to push your filter work if so.

@Jorropo Jorropo merged commit 1bd68fc into master Sep 30, 2020
@Jorropo Jorropo deleted the feat/rememberTheme branch September 30, 2020 12:05
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✅ Approved Pull Request has been approved and can be merged labels Sep 30, 2020
@github-actions
Copy link

🎉 This PR is included in version 2.89.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ✨ Merged Pull Request has been merged successfully released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants