Skip to content
This repository has been archived by the owner on Jun 23, 2021. It is now read-only.

Sync sys darkmode #70

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

Nuage21
Copy link
Member

@Nuage21 Nuage21 commented Feb 4, 2021

Changes

When entering the site for the first time (the dark mode has never been set using the toggle button), it automatically detects the device's appearance mode (light or dark) and applies it to the site, it'll ignore the device mode if the user has already visited the site.

How Has This Been Tested?

This has been tested on 2 devices Macbook/iPhone with Safari/Chrome, the code(device's mode detector) is supported on all browsers.

Mentions

@wrussell1999 @ShrillShrestha

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • [ x] New feature and improvements (non-breaking change which adds/improves functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Code Refactoring (internal change which is not user facing)
  • Documentation
  • Test, CI, or build

Checklist

  • [x ] Coding style of this project is followed
  • Change in documentation is required
  • Documentation has been updated accordingly
  • Unit tests have been added to cover code change

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant