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

Updates to Fireproofing dialogue and Settings #1044

Merged
merged 7 commits into from
Feb 2, 2022

Conversation

amddg44
Copy link
Contributor

@amddg44 amddg44 commented Feb 1, 2022

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

Description:
This PR updates copy for Fireproof dialogue and Settings

Steps to test this PR:

  1. From the home screen, login to a website that will trigger the Fireproof prompt suggestion alert dialogue e.g. amazon.com. Confirm dialogue now contains a title and message, and updated FIreproof button title as per design
  2. Via the overflow menu, select the 'Fireproof This Site' option and confirm it shows the same updated alert dialogue as in Step 1
  3. In Settings -> Fireproof Sites, confirm section header text above the toggle has been removed and that the toggle copy is correct. Also confirm that the section header text for the list of websites is correct
  4. Remove all fireproof websites (if any) and confirm that when there are no Fireproof sites saved that placeholder copy is 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 13
  • 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 amddg44 requested a review from miasma13 February 1, 2022 18:30
@miasma13 miasma13 self-assigned this Feb 1, 2022
@amddg44 amddg44 changed the title Feature/anya/fireproofing copy updates Updates to Fireproofing dialogue and Settings Feb 1, 2022
Copy link
Contributor

@miasma13 miasma13 left a comment

Choose a reason for hiding this comment

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

Why is the whole DuckDuckGo/en.lproj/Localizable.strings removed?

Copy link
Contributor

@miasma13 miasma13 left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@amddg44 amddg44 merged commit 26d8641 into develop Feb 2, 2022
@amddg44 amddg44 deleted the feature/anya/fireproofing-copy-updates branch February 2, 2022 16:38
samsymons added a commit that referenced this pull request Feb 4, 2022
* develop:
  Fix a typo in the waitlist copy. (#1045)
  Only reset lastUpgradedURL if loading new URL comes from the user action (#1046)
  Updates to Fireproofing dialogue and Settings  (#1044)
  Remove CODEOWNERS.
  Fix bookmarks edit button when data source empty (#1043)
  Fix Fingerprinting UI tests. (#1041)
  Bump version number
  Add JS Alert block button (#1039)
  Update CONTRIBUTING.md
  Fix back button doesn't trigger back action (#1027)
  Set version to 7.65.6.
  Widget crash fix (#1038)
  Add keychain items to debug menu (#1037)
  Brad/tracking params (#1016)
  Prevent repeated completions. Respect feature flags (#1036)
  Update release notes
  Change build version to 0
  Update version number
  Add Anya to CODEOWNERS. (#1033)
  fix AMP related crash (#1032)
Bunn pushed a commit that referenced this pull request Feb 4, 2022
* Fireproof copy update for alert

* Fireproof copy updates to Fireproof Settings

* Additional Fireproof copy update

* Translations updated

* Multiline support for Fireproof toggle label to accommodate long translations

* Contents of Localizable.strings fixed
samsymons added a commit that referenced this pull request Feb 8, 2022
# By Sam Symons (6) and others
# Via Bartek Waresiak (2) and others
* develop:
  Fix a typo in the waitlist copy. (#1045)
  Only reset lastUpgradedURL if loading new URL comes from the user action (#1046)
  Updates to Fireproofing dialogue and Settings  (#1044)
  Remove CODEOWNERS.
  Fix bookmarks edit button when data source empty (#1043)
  Fix Fingerprinting UI tests. (#1041)
  Bump version number
  Add JS Alert block button (#1039)
  Update CONTRIBUTING.md
  Fix back button doesn't trigger back action (#1027)
  Set version to 7.65.6.
  Widget crash fix (#1038)
  Add keychain items to debug menu (#1037)
  Brad/tracking params (#1016)
  Prevent repeated completions. Respect feature flags (#1036)

# Conflicts:
#	DuckDuckGo/UserText.swift
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.

3 participants