Skip to content

Commit a9c17b5

Browse files
committed
✨ create settings section header component
1 parent 181e832 commit a9c17b5

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

client/components/mobile/Selector.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { prop } from '../../theme';
55

66

77
const PreferenceTitle = styled.h4.attrs(props => ({ ...props, className: 'preference__title' }))`
8-
color: ${prop('primaryTextColor')} !important;
8+
color: ${prop('primaryTextColor')};
99
`;
1010

1111
const Preference = styled.div.attrs(props => ({ ...props, className: 'preference' }))`

client/modules/Mobile/MobilePreferences.jsx

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import Screen from '../../components/mobile/MobileScreen';
1212
import Header from '../../components/mobile/Header';
1313
import Selector from '../../components/mobile/Selector';
1414
import { ExitIcon } from '../../common/icons';
15-
import { remSize } from '../../theme';
15+
import { remSize, prop } from '../../theme';
1616

1717
const IconLinkWrapper = styled(Link)`
1818
width: 3rem;
@@ -30,12 +30,17 @@ const SettingsHeader = styled(Header)`
3030
background: transparent
3131
`;
3232

33+
const SectionHeader = styled.h2`
34+
color: ${prop('primaryTextColor')};
35+
padding-top: 2rem
36+
`;
37+
3338

3439
const MobilePreferences = (props) => {
3540
const { setTheme, setAutosave, setLinewrap } = props;
3641
const { theme, autosave, linewrap } = props;
3742

38-
const preferences = [
43+
const generalSettings = [
3944
{
4045
title: 'Theme',
4146
value: theme,
@@ -95,12 +100,14 @@ const MobilePreferences = (props) => {
95100
<IconLinkWrapper to="/mobile" aria-label="Return to ide view">
96101
<ExitIcon />
97102
</IconLinkWrapper>
98-
99103
</div>
100104
</SettingsHeader>
101105
<section className="preferences">
102106
<Content>
103-
{ preferences.map(option => <Selector key={`${option.title}wrapper`} {...option} />) }
107+
<SectionHeader>General Settings</SectionHeader>
108+
{ generalSettings.map(option => <Selector key={`${option.title}wrapper`} {...option} />) }
109+
110+
<SectionHeader>Accessibility</SectionHeader>
104111
</Content>
105112
</section>
106113
</section>

0 commit comments

Comments
 (0)