fix: pr review - use isDarkTheme everywhere

pull/2522/head
William Grant 3 years ago
parent de94d2e64a
commit 702cef6565

@ -4,7 +4,7 @@ import styled from 'styled-components';
// @ts-ignore // @ts-ignore
import Picker from '@emoji-mart/react'; import Picker from '@emoji-mart/react';
import { useSelector } from 'react-redux'; import { useSelector } from 'react-redux';
import { getTheme } from '../../state/selectors/theme'; import { getTheme, isDarkTheme } from '../../state/selectors/theme';
import { FixedBaseEmoji, FixedPickerProps } from '../../types/Reaction'; import { FixedBaseEmoji, FixedPickerProps } from '../../types/Reaction';
import { import {
ColorsType, ColorsType,
@ -101,6 +101,7 @@ export const SessionEmojiPanel = forwardRef<HTMLDivElement, Props>((props: Props
const { onEmojiClicked, show, isModal = false, onKeyDown } = props; const { onEmojiClicked, show, isModal = false, onKeyDown } = props;
const primaryColor = useSelector(getPrimaryColor); const primaryColor = useSelector(getPrimaryColor);
const theme = useSelector(getTheme); const theme = useSelector(getTheme);
const isDarkMode = useSelector(isDarkTheme);
const [panelBackgroundRGB, setPanelBackgroundRGB] = useState(''); const [panelBackgroundRGB, setPanelBackgroundRGB] = useState('');
const [panelTextRGB, setPanelTextRGB] = useState(''); const [panelTextRGB, setPanelTextRGB] = useState('');
@ -139,7 +140,7 @@ export const SessionEmojiPanel = forwardRef<HTMLDivElement, Props>((props: Props
ref={ref} ref={ref}
> >
<Picker <Picker
theme={theme.includes('light') ? 'light' : 'dark'} theme={isDarkMode ? 'dark' : 'light'}
i18n={i18nEmojiData} i18n={i18nEmojiData}
onEmojiSelect={onEmojiClicked} onEmojiSelect={onEmojiClicked}
onKeyDown={onKeyDown} onKeyDown={onKeyDown}

@ -47,7 +47,7 @@ import { UserUtils } from '../../session/utils';
import { getLatestReleaseFromFileServer } from '../../session/apis/file_server_api/FileServerApi'; import { getLatestReleaseFromFileServer } from '../../session/apis/file_server_api/FileServerApi';
import { switchThemeTo } from '../../themes/switchTheme'; import { switchThemeTo } from '../../themes/switchTheme';
import { ThemeStateType } from '../../themes/constants/colors'; import { ThemeStateType } from '../../themes/constants/colors';
import { getTheme } from '../../state/selectors/theme'; import { isDarkTheme } from '../../state/selectors/theme';
const Section = (props: { type: SectionType }) => { const Section = (props: { type: SectionType }) => {
const ourNumber = useSelector(getOurNumber); const ourNumber = useSelector(getOurNumber);
@ -55,7 +55,7 @@ const Section = (props: { type: SectionType }) => {
const dispatch = useDispatch(); const dispatch = useDispatch();
const { type } = props; const { type } = props;
const theme = useSelector(getTheme); const isDarkMode = useSelector(isDarkTheme);
const focusedSection = useSelector(getFocusedSection); const focusedSection = useSelector(getFocusedSection);
const isSelected = focusedSection === props.type; const isSelected = focusedSection === props.type;
@ -65,9 +65,9 @@ const Section = (props: { type: SectionType }) => {
dispatch(editProfileModal({})); dispatch(editProfileModal({}));
} else if (type === SectionType.ColorMode) { } else if (type === SectionType.ColorMode) {
const currentTheme = String(window.Events.getThemeSetting()); const currentTheme = String(window.Events.getThemeSetting());
const newTheme = (currentTheme.includes('light') const newTheme = (isDarkMode
? currentTheme.replace('light', 'dark') ? currentTheme.replace('dark', 'light')
: currentTheme.replace('dark', 'light')) as ThemeStateType; : currentTheme.replace('light', 'dark')) as ThemeStateType;
// We want to persist the primary color when using the color mode button // We want to persist the primary color when using the color mode button
await switchThemeTo({ await switchThemeTo({
@ -137,7 +137,7 @@ const Section = (props: { type: SectionType }) => {
return ( return (
<SessionIconButton <SessionIconButton
iconSize="medium" iconSize="medium"
iconType={theme.includes('light') ? 'sun' : 'moon'} iconType={isDarkMode ? 'moon' : 'sun'}
dataTestId="theme-section" dataTestId="theme-section"
notificationCount={unreadToShow} notificationCount={unreadToShow}
onClick={handleClick} onClick={handleClick}

Loading…
Cancel
Save