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.

Sign up for GitHub

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

Jump to bottom

Update Settings design: Add new SettingsListItem #5318

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Update Settings design: Add new SettingsListItem #5318

wants to merge 7 commits into from

Commits on Nov 27, 2024

  1. duplicating content settings and privacy

    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    57c5d5a View commit details
    Browse the repository at this point in the history

  2. add new icons

    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    81a615b View commit details
    Browse the repository at this point in the history

  3. add new SettingsListItem

    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    0a702a2 View commit details
    Browse the repository at this point in the history

  4. switch to SettingsListItem in privacy section 

    Also adding all the new icons, removing secondary text and itemState, and adding the new isOn for search and web tracking settings

Next commit we'll ensure the indicators are on/off
    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    628f8d0 View commit details
    Browse the repository at this point in the history

  5. set items to on/off based on state 

    I've removed appTrackingProtectionOnboardingShown state as only have 2 states now, on/off, but I will follow up and ask Thomas if this is expected
    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    bc4bb02 View commit details
    Browse the repository at this point in the history

  6. add SettingsListItem to ADS 

    I think this is worth having on the ADS screen, even if not officially a component. Happy to move it if needs be.
    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    1a7d27c View commit details
    Browse the repository at this point in the history

  7. formatting

    @mikescamell
    mikescamell committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    38b133a View commit details
    Browse the repository at this point in the history