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

add popupmenu in main screen, move theme change in popupmenu #103

Merged
merged 4 commits into from
Jun 22, 2021

Conversation

deep-diver
Copy link
Member

@deep-diver deep-diver commented Jun 22, 2021

Screen Shot 2021-06-22 at 11 10 48 AM

context.watch<CustomTheme>().isDarkMode didn't work, so I changed it to context.read<CustomTheme>().isDarkMode

@deep-diver deep-diver requested a review from kkweon June 22, 2021 02:13
@codecov
Copy link

codecov bot commented Jun 22, 2021

Codecov Report

Merging #103 (b561b29) into main (54396eb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #103   +/-   ##
=======================================
  Coverage   79.06%   79.06%           
=======================================
  Files           4        4           
  Lines          86       86           
=======================================
  Hits           68       68           
  Misses         17       17           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 54396eb...b561b29. Read the comment docs.

Copy link
Member

@kkweon kkweon left a comment

Choose a reason for hiding this comment

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

오오오 very good 입니다

@kkweon kkweon enabled auto-merge (squash) June 22, 2021 04:22
@kkweon kkweon merged commit 11dd951 into main Jun 22, 2021
@kkweon kkweon deleted the add-popupmenu branch June 22, 2021 04:31
@github-actions
Copy link

🎉 This PR is included in version 1.10.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants