Skip to content

Conversation

@wingsum93
Copy link
Contributor

@wingsum93 wingsum93 commented Apr 21, 2023

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.

Pre-launch Checklist

  • I read the rules PR (title, labels, commit messages...).
  • I definitely don't PR files with pointless edits.
  • I added screenshots if change UI/UX (has modified code in lib/src/ui/*).
  • I writen all test cases coverage and all passed.
  • All existing and new tests are passing.

Testcases covered (check if passed)

  • Manual test change theme by clicking icon

Screenshots (if change UI/UX)

@wingsum93
Copy link
Contributor Author

This PR fixed the issue in #5

Copy link
Owner

@lambiengcode lambiengcode left a comment

Choose a reason for hiding this comment

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

lgtm!

@lambiengcode lambiengcode merged commit d93c496 into lambiengcode:main Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants