[PM-16428] Option for primary click action to autofill on Vault view

[kspearrin]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-16428

📔 Objective

• Add an option/setting for the primary action of autofill suggestions on the Vault view to perform an "Autofill" operation instead of the default "View" action.
• "View" becomes an option under the 3 dot "more options" menu.
• "Fill" button goes away.

📸 Screenshots

New autofill setting

Vault view

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[github-actions]

Checkmarx One – Scan Summary & Details – c40870bb-f385-453c-acdd-8e0f244ae1ad

New Issues

Severity	Issue	Source File / Package	Checkmarx Insight
	CVE-2024-11395	Npm-electron-33.2.1	Vulnerable Package
	Client_Privacy_Violation	/apps/browser/src/autofill/popup/settings/autofill.component.html: 40	Attack Vector
	Client_Privacy_Violation	/apps/browser/src/autofill/popup/settings/autofill.component.html: 107	Attack Vector

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.

Project coverage is 33.77%. Comparing base (0619ef5) to head (05fd131).

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...-list-items/autofill-vault-list-items.component.ts	0.00%	7 Missing ⚠️
...2/item-more-options/item-more-options.component.ts	0.00%	4 Missing ⚠️
.../services/vault-settings/vault-settings.service.ts	0.00%	4 Missing ⚠️
.../src/autofill/popup/settings/autofill.component.ts	0.00%	3 Missing ⚠️
...c/vault/services/key-state/vault-settings.state.ts	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12557      +/-   ##
==========================================
- Coverage   33.78%   33.77%   -0.01%     
==========================================
  Files        2912     2912              
  Lines       90693    90710      +17     
  Branches    17151    17153       +2     
==========================================
  Hits        30641    30641              
- Misses      57666    57683      +17     
  Partials     2386     2386              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jprusik]

LGTM!

(smoke-tested as well; everything behaves as expected)