feat(redux-devtools-app): use prefers-color-scheme to set theme.light if user has not set a preferred theme

This commit is contained in:
FaberVitale 2021-09-13 22:53:06 +02:00
parent a9ef668d9b
commit b3250c068e
7 changed files with 96 additions and 8 deletions

View File

@ -8,7 +8,10 @@ import configureStore from '../../../app/stores/panelStore';
import '../../views/devpanel.pug';
import { Action, Store } from 'redux';
import { StoreAction } from '@redux-devtools/app/lib/actions';
import {
applyMediaFeaturesPreferences,
StoreAction,
} from '@redux-devtools/app/lib/actions';
import { PanelMessage } from '../../../app/middlewares/api';
import { StoreStateWithoutSocket } from '../../../app/reducers/panel';
import { PersistGate } from 'redux-persist/integration/react';
@ -33,9 +36,20 @@ function renderDevTools() {
unmountComponentAtNode(node!);
clearTimeout(naTimeout);
({ store, persistor } = configureStore(position, bgConnection));
const onBeforeLift = () => {
if (store) {
store.dispatch(applyMediaFeaturesPreferences());
}
};
render(
<Provider store={store}>
<PersistGate loading={null} persistor={persistor}>
<PersistGate
loading={null}
persistor={persistor}
onBeforeLift={onBeforeLift}
>
<App position={position} />
</PersistGate>
</Provider>,

View File

@ -8,6 +8,7 @@ import configureStore from '../../../app/stores/windowStore';
import { MonitorMessage } from '../../../app/middlewares/api';
import '../../views/window.pug';
import { applyMediaFeaturesPreferences } from '@redux-devtools/app/lib/actions';
const position = location.hash;
@ -25,9 +26,17 @@ chrome.runtime.getBackgroundPage((window) => {
bg.onMessage.addListener(update);
update();
const onBeforeLift = () => {
localStore.dispatch(applyMediaFeaturesPreferences());
};
render(
<Provider store={localStore}>
<PersistGate loading={null} persistor={persistor}>
<PersistGate
loading={null}
persistor={persistor}
onBeforeLift={onBeforeLift}
>
<App position={position} />
</PersistGate>
</Provider>,

View File

@ -3,6 +3,7 @@ import { AuthStates, States } from 'socketcluster-client/lib/scclientsocket';
import {
CHANGE_SECTION,
CHANGE_THEME,
APPLY_MEDIA_FEATURES_PREFERENCES,
SELECT_INSTANCE,
SELECT_MONITOR,
UPDATE_MONITOR_STATE,
@ -44,9 +45,9 @@ import {
import { Action } from 'redux';
import { Features, State } from '../reducers/instances';
import { MonitorStateMonitorState } from '../reducers/monitor';
import { LiftedAction } from '@redux-devtools/core';
import { LiftedAction, LiftedState } from '@redux-devtools/core';
import { Data } from '../reducers/reports';
import { LiftedState } from '@redux-devtools/core';
import { prefersDarkColorScheme } from '../utils/media-queries';
let monitorReducer: (
monitorProps: unknown,
@ -77,10 +78,29 @@ export interface ChangeThemeAction {
readonly scheme: Scheme;
readonly dark: boolean;
}
export interface ApplyMediaFeaturesPreferencesAction {
readonly type: typeof APPLY_MEDIA_FEATURES_PREFERENCES;
readonly prefersDarkColorScheme: boolean;
}
export function changeTheme(data: ChangeThemeData): ChangeThemeAction {
return { type: CHANGE_THEME, ...data.formData };
}
/**
* @see https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries#media_features
*/
export function applyMediaFeaturesPreferences(
payload?: Partial<Omit<ApplyMediaFeaturesPreferencesAction, 'type'>>
): ApplyMediaFeaturesPreferencesAction {
return {
prefersDarkColorScheme: prefersDarkColorScheme(),
...payload,
type: APPLY_MEDIA_FEATURES_PREFERENCES,
};
}
export interface InitMonitorAction {
type: '@@INIT_MONITOR';
newMonitorState: unknown;
@ -564,6 +584,7 @@ export interface ErrorAction {
export type StoreActionWithoutUpdateStateOrLiftedAction =
| ChangeSectionAction
| ChangeThemeAction
| ApplyMediaFeaturesPreferencesAction
| MonitorActionAction
| SelectInstanceAction
| SelectMonitorAction

View File

@ -1,5 +1,7 @@
export const CHANGE_SECTION = 'main/CHANGE_SECTION';
export const CHANGE_THEME = 'main/CHANGE_THEME';
export const APPLY_MEDIA_FEATURES_PREFERENCES =
'main/APPLY_MEDIA_FEATURES_PREFERENCES';
export const UPDATE_STATE = 'devTools/UPDATE_STATE';
export const SET_STATE = 'devTools/SET_STATE';

View File

@ -7,7 +7,7 @@ import configureStore from './store/configureStore';
import { CONNECT_REQUEST } from './constants/socketActionTypes';
import App from './containers/App';
import { StoreState } from './reducers';
import { StoreAction } from './actions';
import { StoreAction, applyMediaFeaturesPreferences } from './actions';
class Root extends Component {
store?: Store<StoreState, StoreAction>;
@ -27,11 +27,26 @@ class Root extends Component {
this.persistor = persistor;
}
/**
* @hidden
* @private
*/
private _checkMediaFeaturesPreferences = () => {
if (this.store) {
this.store.dispatch(applyMediaFeaturesPreferences());
}
};
render() {
if (!this.store) return null;
return (
<Provider store={this.store}>
<PersistGate loading={null} persistor={this.persistor!}>
<PersistGate
loading={null}
persistor={this.persistor!}
onBeforeLift={this._checkMediaFeaturesPreferences}
>
<App />
</PersistGate>
</Provider>

View File

@ -1,11 +1,12 @@
import { Scheme, Theme } from '@redux-devtools/ui';
import { CHANGE_THEME } from '../constants/actionTypes';
import { CHANGE_THEME, APPLY_MEDIA_FEATURES_PREFERENCES, } from '../constants/actionTypes';
import { StoreAction } from '../actions';
export interface ThemeState {
readonly theme: Theme;
readonly scheme: Scheme;
readonly light: boolean;
readonly latestChangeBy?: string;
}
export default function theme(
@ -21,7 +22,20 @@ export default function theme(
theme: action.theme,
scheme: action.scheme,
light: !action.dark,
latestChangeBy: CHANGE_THEME,
};
}
if (
action.type === APPLY_MEDIA_FEATURES_PREFERENCES &&
state.latestChangeBy !== CHANGE_THEME
) {
return {
...state,
light: !action.prefersDarkColorScheme,
latestChangeBy: APPLY_MEDIA_FEATURES_PREFERENCES,
};
}
return state;
}

View File

@ -0,0 +1,13 @@
/**
* @see https://developer.mozilla.org/en-US/docs/Web/CSS/@media/prefers-color-scheme
*/
export function prefersDarkColorScheme(): boolean {
if (
typeof window !== 'undefined' &&
typeof window.matchMedia === 'function'
) {
return window.matchMedia('(prefers-color-scheme: dark)').matches;
}
return false;
}