Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@afterxleep
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1204099484721401/1202300800602708/f

Description:

  • Adds automatically generated letter icons to Autofill logins when no favicon is present

Steps to test this PR:

  1. Fire up the Logins (Autofill UI)
  2. Add a login for a website you have not visited before (Leave title empty)
  3. Observe an icon with the first two letters of the website is created
  4. Change the site URL to be a subdomain (i.e. accounts.google.com)
  5. Observe the icon displays the first two letters (i.e. Go for accounts.google.com)
  6. Add a title to the created login (i.e. Test Login)
  7. Observe the icon now displays the first two letters of the title
  8. Observe that as you change the domain, another color icon is picked automatically
Screenshot 2023-08-21 at 5 47 02 PM
Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

afterxleep and others added 30 commits August 9, 2023 20:04
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/Common/View/SwiftUI/LoginFaviconView.swift
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@afterxleep afterxleep requested a review from amddg44 August 23, 2023 13:23
@afterxleep afterxleep changed the title Daniel/autofill icon letters Autofill UI letter icons Aug 23, 2023
@afterxleep afterxleep merged commit 8fdf0e8 into develop Aug 23, 2023
@afterxleep afterxleep deleted the daniel/autofill-icon-letters branch August 23, 2023 14:25
samsymons added a commit that referenced this pull request Aug 23, 2023
# By Diego Rey Mendez (7) and others
# Via Sam Symons (2) and others
* develop: (26 commits)
  Improve Sync-related database cleaning logic (#1529)
  Update onboarding-related error states (#1504)
  Prevents launching our menu agent without an auth code. (#1516)
  Autofill UI letter icons (#1535)
  Cleans up some code (#1517)
  Revert "Autofill Letter Icons" (#1534)
  Adds remote pre-commit installer, which includes automatic fix for linter (#1369)
  Autofill Letter Icons (#1475)
  change context menu for mailto links (#1513)
  Updates the version to 1.53.1
  Updated the embedded files for 1.53.1
  Update the phased rollout tester to avoid caching the config (#1520)
  Require Duck Player scheme URL to be passed from YouTube Overlay User Script (#1519)
  Add pixels related to Duck Player usage (#1515)
  only allow error reloads on http(s) urls (#1523)
  Standardize TDS Loading Error handling (#1524)
  Move pixel sender logic into the main view controller (#1528)
  Update the phased rollout tester to avoid caching the config (#1520)
  Set version to 1.52.3.
  Move pixel sender logic into the main view controller (#1528)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/AppDelegate/AppDelegate.swift
#	DuckDuckGo/Common/Localizables/UserText.swift
#	DuckDuckGo/Common/Utilities/UserDefaultsWrapper.swift
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant