-
Notifications
You must be signed in to change notification settings - Fork 139
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
Update widget design #1343
Update widget design #1343
Conversation
Generated by 🚫 Danger |
177315b
to
fd86fef
Compare
So I'm not sure it is expected, but I'm also not sure we're doing anything special in order to get them to appear there. It is just leveraging whatever iOS already does to make widgets available on desktop. |
@@ -0,0 +1,122 @@ | |||
import SwiftUI | |||
|
|||
let widgetRedDark = Color(red: 217, green: 32, blue: 28) |
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.
Instead of creating global constants you can do the following:
Extension Color {
static let widgetRedDark = Color(red: 217, green: 32, blue: 28)
}
And then in any place you need it you just do: .widgetRedDark
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.
Done in 36762aa
I faced the same while testing with the old widget code. As @jgcaruso mentioned I don't think we do anything special... |
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.
From my checks on iOS17 it's all looking and code is now updated with my suggestions.
So this is a green for me, but lets wait for @bjtitus and @leandroalonso to approve too.
I'm gonna test this a bit more but I propose we leave that for 7.68 given 7.67 has already some important changes. |
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.
Sure thing! How will I know it is safe to merge? Alternatively, feel free to merge this in when appropriate if there are no other requested changes. I'm not planning any additional changes at this point. Only thing planned at this point is a separate PR once this is merged in for the new medium size "Now Playing" widget |
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.
Looks good 👍🏻
Thank you so much for all of your hard work and being receptive of our feedback @jgcaruso! 🙇
I just merged this for the 7.68 release now that 7.67 has been frozen. |
@bjtitus I believe we should add this to the |
@leandroalonso 👍 Submitted in #1864 |
Updates to widgets based on new designs Lkt7fuf9Nq3XvfAFPCfpTD-fi-995_4119
Also introduces a
PCWidgetColorScheme
struct to store different collections of colours to be passed to the different versions of the widgets to support a new style and a "contrast" style that is more like the current widgets.cc @david-gonzalez-a8c
Some screenshots
The widget chooser
Screen.Recording.2024-01-04.at.4.57.03.PM.mov
iPhone 13 mini in dark mode (widget doesn't change based on light/dark mode)
iPhone 15
iPhone 15 - other widgets
Desktop widgets
ios 16 non-interactive
To test
Test on an iOS 17 device (for interactive widgets) and a lower iOS version (for non interactive widgets)
Test on an iOS 17 device, linked to a MacOS Sonoma mac for Desktop widgets
System Settings ➝ Desktop & Dock
, and in the "Widgets" section, click the iPhone dropdownUpdate experience
Checklist
CHANGELOG.md
if necessary.