-
Notifications
You must be signed in to change notification settings - Fork 424
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
Password Manager widget and app shortcut #2619
Conversation
…d-mgr-shortcut # Conflicts: # DuckDuckGo/AppDelegate.swift # Widgets/WidgetViews.swift
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.
Validated and working as expected. Looks really awesome @amddg44, nice job! 🚀 Just left too super minor suggestions.
@@ -349,6 +356,20 @@ final class AutofillLoginSettingsListViewController: UIViewController { | |||
} | |||
} | |||
|
|||
private func openSearchIfRequired() { | |||
// Don't auto open search if user has selected an account | |||
if selectedAccount != nil { |
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.
Super minor suggestion - If you preferred, we could use a guard
here, e.g:
if selectedAccount != nil { | |
guard selectedAccount == nil else { return } |
Widgets/WidgetViews.swift
Outdated
|
||
VStack(alignment: .center, spacing: 14) { | ||
|
||
Image("WidgetPasswordIllustration") |
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.
Image("WidgetPasswordIllustration") | |
Image(.widgetPasswordIllustration) |
Think you could use that here (and other places too if so inclined)
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.
Translations LGTM
# By Mariusz Śpiewak (4) and others # Via Chris Brind (1) and others * main: Reverting accidental push to main (#2718) Add SubscriptionContainerViewModel and Manually hide loader + Pixel (#2687) Release 7.115.0-2 (#2712) soft revert history suggestions (#2711) Bring back accessibility identifiers for onboarding buttons (#2709) BSK release 133.1.0 (#2708) Password Manager widget and app shortcut (#2619) Release 7.115.0-1 (#2707) Update set-as-default onboarding illustration for dark mode (#2694) update app store prompt logic (#2678) Fix status bar color on regular width size class (#2705) Updated settings (#2603) Update BSK with autofill 11.0.1 (#2704) # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Task/Issue URL: https://app.asana.com/0/42792087274227/1206508535741153/f
Tech Design URL:
CC:
Description:
Adds home screen widget, lock screen widget & app shortcut to quickly access passwords
Steps to test this PR:
Search Passwords
Copy Testing:
’
rather than'
Orientation Testing:
Device Testing:
OS Testing:
Theme Testing:
Internal references:
Software Engineering Expectations
Technical Design Template