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

Address autofill security concerns with copy changes #3211

Merged
merged 31 commits into from
Sep 19, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
ef2bca8
Add lock asset
graeme Aug 16, 2024
bc9faf9
Replace passwords subtitle with explainer view
graeme Aug 16, 2024
fd3f8ce
Add wip empty state text
graeme Aug 23, 2024
6da6b83
Show security info on clicking lock
graeme Sep 3, 2024
6b018eb
Fix empty state text
graeme Sep 4, 2024
104aee4
Update password explainer text
graeme Sep 4, 2024
3a39fd1
Add logic for showing import password explainer
graeme Sep 4, 2024
f8edc47
Hide security button based on autolock preferences
graeme Sep 4, 2024
5bf59bf
Make autolock off copy changes from Windows SR
graeme Sep 5, 2024
921da27
Fix crashy popover issue
graeme Sep 5, 2024
c6cb857
Fix settings link handling
graeme Sep 5, 2024
4d4560b
Fix settings link handing on Save info view
graeme Sep 5, 2024
d833a7a
Fix Settings casing
graeme Sep 5, 2024
2b3d5e1
Remove the and from the encrypted passwords message
graeme Sep 10, 2024
270bcda
More copy updates
graeme Sep 11, 2024
32d81a1
+ strings file
graeme Sep 11, 2024
f22aedd
Only show security button border when hovering
graeme Sep 11, 2024
e22b6f1
Make security info popover on hover
graeme Sep 13, 2024
2ce0c7a
Implement feedback on info tooltip view
graeme Sep 16, 2024
4ba21ba
Merge remote-tracking branch 'origin/main' into graeme/address-autofi…
graeme Sep 16, 2024
e857fdc
Remove duplicate string after main merge
graeme Sep 16, 2024
1cddd3b
Reduce auto-dismiss time
graeme Sep 16, 2024
cfbdd3d
Update lock hover view colours again
graeme Sep 17, 2024
e075a37
Fix dark mode by converting view to box
graeme Sep 18, 2024
cb88fc3
Lazy load the autolock var
graeme Sep 19, 2024
d0b7ece
Merge remote-tracking branch 'origin/main' into graeme/address-autofi…
graeme Sep 19, 2024
64e0d24
Use factory for autofill prefs to fix tests
graeme Sep 19, 2024
546e4d2
Revert "Use factory for autofill prefs to fix tests"
graeme Sep 19, 2024
93b1477
Revert "Lazy load the autolock var"
graeme Sep 19, 2024
2b10f74
Prevent Database assertion failure in dependencies
graeme Sep 19, 2024
664fb26
Prepend lock to empty screen message
graeme Sep 19, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
More copy updates
  • Loading branch information
graeme committed Sep 11, 2024
commit 270bcdac2f612967f48fa38f562a8f277071c2be
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ extension UserText {
value: "Passwords are encrypted. Nobody but you can see them, not even us.",
comment: "Label for default empty state description")
static let pmEmptyStateDefaultDescriptionAutolockOff = NSLocalizedString("pm.empty.default.description.extended.v2",
value: "Passwords are encrypted. We recommend setting up Auto-lock to keep your passwords even more secure.",
value: "Passwords are encrypted.",
comment: "Label for default empty state description when the autolock feature is off")
static let pmEmptyStateLearnMoreLink = NSLocalizedString("pm.empty.learn.more.link", value: "Learn more", comment: "Text for link to learn more about DuckDuckGo password manager")
static let pmEmptyStateDefaultButtonTitle = NSLocalizedString("pm.empty.default.button.title", value: "Import Passwords", comment: "Import passwords button title for default empty state")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -288,11 +288,11 @@ final class PasswordManagementItemListModel: ObservableObject {
}

var emptyStateMessageLinkText: String {
autofillPreferences.isAutoLockEnabled ? UserText.learnMore : UserText.goToSettings
UserText.learnMore
}

var emptyStateMessageLinkURL: URL {
autofillPreferences.isAutoLockEnabled ? URL.passwordManagerLearnMore : URL.settingsPane(.autofill)
URL.passwordManagerLearnMore
}

@Published private(set) var emptyState: EmptyState = .none
Expand Down
Loading