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

Apply max-width constraints to autofill dialogs' contents #3773

Conversation

CDRussell
Copy link
Member

@CDRussell CDRussell commented Oct 30, 2023

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

Description

We want to apply a max-width constraint to the contents of the autofill dialogs. Some of the autofill dialogs had different ways of constraining content before, and others had none; this PR consolidates them all into using this approach:

  • Guideline left set to 10% from left edge of dialog, guideline right set to 10% from right edge of dialog
  • Autofill dialog content can use the remaining 80% of dialog width, up to max of 480dp

ℹ️ Note for reviewing the PR:

  • a lot of the changes are caused by indentation as views are now inside another view group
  • tip: hide whitespace differences to see only what's actually changed, ignoring the change indentation
Screenshot 2023-10-31 at 12 34 27

Steps to test this PR

Save login prompt

Select login prompt

Update login prompt

  • Visit https://fill.dev/form/login-simple again and dismiss any prompts to use a saved login
  • Instead, enter same username as before abcd and a new password
  • Verify the update login prompt looks ok

Generate password prompt

Email protection in-context signup

  • Visit https://fill.dev/form/registration-email when you're not signed into email protection (requires device to be in English locale)
  • Tap on grey Dax in email field; verify the prompt to sign up for email protection looks ok

Email protection, signed in state

  • Sign into email protection (doesn't matter how you sign in, in-context or normal approach from settings)
  • Visit https://fill.dev/form/registration-email again, and tap on email field
  • Verify the prompt to use Email Protection looks ok

UI changes

Screenshots available here for phones and tablets, portrait and landscape

@CDRussell CDRussell force-pushed the feature/craig/autofill_max_width_on_prompts branch from 882733a to ee077a0 Compare October 30, 2023 11:22
@CDRussell CDRussell force-pushed the feature/craig/autofill_enabled_by_default branch from 41fd748 to 1aae7f9 Compare October 30, 2023 11:30
@CDRussell CDRussell force-pushed the feature/craig/autofill_max_width_on_prompts branch 3 times, most recently from eea6315 to 4a94337 Compare October 31, 2023 12:31
@CDRussell CDRussell force-pushed the feature/craig/autofill_enabled_by_default branch from 1aae7f9 to 9367453 Compare October 31, 2023 12:38
@CDRussell CDRussell force-pushed the feature/craig/autofill_max_width_on_prompts branch from 4a94337 to f84a926 Compare October 31, 2023 12:38
@CDRussell CDRussell marked this pull request as ready for review October 31, 2023 13:08
@joshliebe joshliebe self-assigned this Oct 31, 2023
Copy link
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than screenshots_dialogs.yaml being empty, lgtm!

@CDRussell CDRussell force-pushed the feature/craig/autofill_max_width_on_prompts branch from f84a926 to 8da2f2c Compare October 31, 2023 16:36
@CDRussell CDRussell merged commit 43c169b into feature/craig/autofill_enabled_by_default Oct 31, 2023
@CDRussell CDRussell deleted the feature/craig/autofill_max_width_on_prompts branch October 31, 2023 20:44
CDRussell added a commit that referenced this pull request Nov 1, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 6, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
@CDRussell CDRussell mentioned this pull request Nov 6, 2023
6 tasks
CDRussell added a commit that referenced this pull request Nov 7, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 7, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 8, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 10, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 10, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 14, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

### Description
We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR: 
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">


### Steps to test this PR

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_ 
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok



### UI changes

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
CDRussell added a commit that referenced this pull request Nov 15, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

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

We want to apply a max-width constraint to the contents of the autofill
dialogs. Some of the autofill dialogs had different ways of constraining
content before, and others had none; this PR consolidates them all into
using this approach:

- Guideline left set to 10% from left edge of dialog, guideline right
set to 10% from right edge of dialog
- Autofill dialog content can use the remaining 80% of dialog width, up
to max of `480dp`

ℹ️ Note for reviewing the PR:
- a lot of the _changes_ are caused by indentation as views are now
inside another view group
- tip: hide whitespace differences to see only what's actually changed,
ignoring the change indentation
<img width="249" alt="Screenshot 2023-10-31 at 12 34 27"
src="https://github.com/duckduckgo/Android/assets/1336281/f19f99f0-6b02-4d3a-9810-4d2a4719432c">

_Save login prompt_
- [x] Visit https://fill.dev/form/login-simple and enter username
[`abcd`] and password (>= 4 chars)
- [x] Verify save prompt looks ok

_Select login prompt_
- [x] Visit https://fill.dev/form/login-simple again
- [x] verify prompt to login with saved credential looks ok

_Update login prompt_
- [x] Visit https://fill.dev/form/login-simple again and dismiss any
prompts to use a saved login
- [x] Instead, enter same username as before `abcd` and a new password
- [x] Verify the update login prompt looks ok

_Generate password prompt_
- [x] Visit https://fill.dev/form/registration-email and tap on password
field
- [x] Verify generate password prompts looks ok

_Email protection in-context signup_
- [x] Visit https://fill.dev/form/registration-email when you're **not**
signed into email protection (requires device to be in English locale)
- [x] Tap on grey Dax in email field; verify the prompt to sign up for
email protection looks ok

_Email protection, signed in state_
- [x] Sign into email protection (doesn't matter how you sign in,
in-context or normal approach from settings)
- [x] Visit https://fill.dev/form/registration-email again, and tap on
email field
- [x] Verify the prompt to use Email Protection looks ok

Screenshots available
[here](https://app.asana.com/0/0/1205842983551006/f) for phones and
tablets, portrait and landscape
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants