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

Autofill Settings layout improvements for landscape / iPad #1328

Merged
merged 7 commits into from Oct 7, 2022

Conversation

amddg44
Copy link
Contributor

@amddg44 amddg44 commented Oct 3, 2022

Task/Issue URL: https://app.asana.com/0/72649045549333/1202809065048067/f
Tech Design URL:
CC:

Description:
Landscape and iPad layout handling for all screens associated with Autofill Settings including:

  • locked view
  • empty autofill view (enabled and disabled states)
  • list view with items and
  • empty search results

Steps to test this PR:

  1. Run through the different view states confirming the layouts match designs for iPhone landscape and iPad portrait & landscape
  2. Confirm iPhone portrait layouts are still correct

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPad

OS Testing:

  • iOS 14
  • iOS 15

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@THISISDINOSAUR
Copy link
Contributor

Still looking at if, but encountered a couple of issues, 1st is no edit button (iPhone 14, iOS 16)
Simulator Screen Shot - iPhone 14 - 2022-10-03 at 18 13 36

And on the same device, I'm encountering a break point when attempting to view an individual login
Screenshot 2022-10-03 at 18 14 53

@amddg44
Copy link
Contributor Author

amddg44 commented Oct 4, 2022

Still looking at if, but encountered a couple of issues, 1st is no edit button (iPhone 14, iOS 16)

Ok, this is one is because Autofill was disabled. I reviewed the designs and confirmed that if Autofill is disabled but user has saved logins there should be an Edit button and have made this change

@amddg44
Copy link
Contributor Author

amddg44 commented Oct 4, 2022

And on the same device, I'm encountering a break point when attempting to view an individual login

This one is strange and I cannot reproduce on my side at all - is this happening for you with all devices? Wondering if it could be a conflict with a previous build that perhaps had notes saved?

@THISISDINOSAUR
Copy link
Contributor

Ah I didn't notice autofill was disabled, the sooner we fix the disabling logic there the better! (I'm working on it atm)
That's a good shout, it's almost certainly because of notes

@THISISDINOSAUR
Copy link
Contributor

Oh actually I have been able to replicate the breakpoint on a new device

Screenshot 2022-10-04 at 11 55 42

@THISISDINOSAUR
Copy link
Contributor

The search bar and section index are different to design, but my bad on the section index for not spotting that the designs aren't platform idiomatic, so don't think we should change it.
The search bar though I'm not sure about. ATM it's the same as in bookmarks, but I looked around a bit for what other apps do and music has a search bar like we have on smaller devices, but it was the only one I could find. WDYT?

Simulator Screen Shot - iPad Air (5th generation) - 2022-10-04 at 11 53 44

@THISISDINOSAUR
Copy link
Contributor

I tried doing a clean build and now the it's gone 🤷‍♀️

@Chaudhary24-cyber
Copy link

Task/Issue URL: https://app.asana.com/0/72649045549333/1202809065048067/f Tech Design URL: CC:

Description: Landscape and iPad layout handling for all screens associated with Autofill Settings including:

  • locked view
  • empty autofill view (enabled and disabled states)
  • list view with items and
  • empty search results

Steps to test this PR:

  1. Run through the different view states confirming the layouts match designs for iPhone landscape and iPad portrait & landscape
  2. Confirm iPhone portrait layouts are still correct

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPad

OS Testing:

  • iOS 14
  • iOS 15

Theme Testing:

  • Light theme
  • Dark theme
Internal references:

Software Engineering Expectations Technical Design Template When ready for review, remember to post the PR in MM

@amddg44
Copy link
Contributor Author

amddg44 commented Oct 5, 2022

The search bar though I'm not sure about. ATM it's the same as in bookmarks, but I looked around a bit for what other apps do and music has a search bar like we have on smaller devices, but it was the only one I could find. WDYT?

Yeah, I go back and forth on this. ATM I'm leaning towards default iOS 16 behaviours and kicking off a conversation on this with design, but don't have a strong opinion either way

@THISISDINOSAUR
Copy link
Contributor

The search bar though I'm not sure about. ATM it's the same as in bookmarks, but I looked around a bit for what other apps do and music has a search bar like we have on smaller devices, but it was the only one I could find. WDYT?

Yeah, I go back and forth on this. ATM I'm leaning towards default iOS 16 behaviours and kicking off a conversation on this with design, but don't have a strong opinion either way

Yeah, that sounds good to me.

Either way, code looks good to me, so I don't see any reason to not merge this if you'd rather before that convo is concluded

@amddg44 amddg44 merged commit fbf7d61 into develop Oct 7, 2022
@amddg44 amddg44 deleted the feature/anya/autofill-settings-ipad-landscape branch October 7, 2022 06:50
bershanskiy pushed a commit to bershanskiy/duckduckgo-iOS that referenced this pull request Jan 13, 2023
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.

None yet

3 participants