diff --git a/webapp/channels/src/components/linking_landing_page/linking_landing_page.tsx b/webapp/channels/src/components/linking_landing_page/linking_landing_page.tsx index 89cf8db8a5..c9c60edf2f 100644 --- a/webapp/channels/src/components/linking_landing_page/linking_landing_page.tsx +++ b/webapp/channels/src/components/linking_landing_page/linking_landing_page.tsx @@ -7,7 +7,6 @@ import {FormattedMessage} from 'react-intl'; import BrowserStore from 'stores/browser_store'; import FormattedMarkdownMessage from 'components/formatted_markdown_message'; -import CheckboxCheckedIcon from 'components/widgets/icons/checkbox_checked_icon'; import desktopImg from 'images/deep-linking/deeplinking-desktop-img.png'; import mobileImg from 'images/deep-linking/deeplinking-mobile-img.png'; @@ -99,12 +98,13 @@ export default class LinkingLandingPage extends PureComponent { return landingPreference && landingPreference === LandingPreferenceTypes.MATTERMOSTAPP; }; - handleChecked = () => { + handleChecked = (e: React.ChangeEvent) => { + this.setState({rememberChecked: e.target.checked}); + // If it was checked, and now we're unchecking it, clear the preference - if (this.state.rememberChecked) { + if (!e.target.checked) { BrowserStore.clearLandingPreference(this.props.siteUrl); } - this.setState({rememberChecked: !this.state.rememberChecked}); }; setPreference = (pref: string, clearIfNotChecked?: boolean) => { @@ -201,16 +201,6 @@ export default class LinkingLandingPage extends PureComponent { this.setState({brandImageError: true}); }; - renderCheckboxIcon = () => { - if (this.state.rememberChecked) { - return ( - - ); - } - - return null; - }; - renderGraphic = () => { const isMobile = UserAgent.isMobile(); @@ -392,18 +382,18 @@ export default class LinkingLandingPage extends PureComponent { /> -
- +
+ {this.renderDownloadLinkSection()} ); diff --git a/webapp/channels/src/sass/routes/_get-app.scss b/webapp/channels/src/sass/routes/_get-app.scss index 71eb8dcba0..43f93db0d7 100644 --- a/webapp/channels/src/sass/routes/_get-app.scss +++ b/webapp/channels/src/sass/routes/_get-app.scss @@ -126,9 +126,10 @@ .get-app__preference { display: flex; margin-top: 23px; + cursor: pointer; > span { - margin-top: -2px; + margin-top: 1px; color: #000; font-family: "Open Sans"; font-size: 16px; @@ -136,27 +137,12 @@ line-height: 22px; opacity: 0.8; } -} -.get-app__checkbox { - width: 18px; - height: 18px; - padding: 0; - border: 2px solid $dark-gray; - border-radius: 3px; - margin-right: 7px; - background: none; - - > * { - pointer-events: none; - } - - &.checked { - border: 0; - - svg { - fill: #145dbf; - } + .get-app__checkbox { + width: 18px; + height: 18px; + padding: 0; + margin-right: 7px; } }