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
Feature: System Color Scheme #5
Conversation
Good stuff! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor comments :]
...tributor.xcodeproj/xcuserdata/tunde.adegoroye.xcuserdatad/xcschemes/xcschememanagement.plist
Outdated
Show resolved
Hide resolved
case .dark: | ||
return "🌚 Dark" | ||
case .system: | ||
return "📱 System" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😍
|
||
final class AppColorSchemeManager: ObservableObject { | ||
|
||
@AppStorage("currentColorScheme") var colorScheme: Int = AppColorScheme.allCases.first!.rawValue // Force unwrap YOLO 🤪 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment 😂
aab4dbb
to
6bcd8d2
Compare
…e app, also did some refactoring on the setting screen to break out views shown in the list.
6bcd8d2
to
f9f74b0
Compare
@adamrushy All those issues should be resolved now 👍🏾 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ✅
Added system settings for light, dark, and system theme for the entire app, also did some refactoring on the setting screen to break out views shown in the list.