diff --git a/app/renderer/components/common/browserButton.js b/app/renderer/components/common/browserButton.js index ee6eb09d22d..8ba8519072e 100644 --- a/app/renderer/components/common/browserButton.js +++ b/app/renderer/components/common/browserButton.js @@ -88,7 +88,14 @@ class NormalizedButton extends ImmutableComponent { data-l10n-args={JSON.stringify(this.props.l10nArgs || {})} data-button-value={this.props.dataButtonValue} onClick={this.props.onClick} - className={css(styles.normalizedButton, this.props.custom)} + className={css( + styles.normalizedButton, + + // For homeButton, stopButton, and bookmarkButton + this.props.navigationButton && styles.normalizedButton_navigationButton, + + this.props.custom + )} // for publisherToggle.js data-test-authorized={this.props.testAuthorized} @@ -202,13 +209,12 @@ const styles = StyleSheet.create({ } }, + // This should be included in navigationBarButtonContainer browserButton_extensionItem: { - WebkitAppRegion: 'no-drag', backgroundSize: 'contain', + backgroundRepeat: 'no-repeat', height: '17px', - margin: '4px 0 0 0', - opacity: '0.85', - backgroundRepeat: 'no-repeat' + opacity: '0.85' }, browserButton_groupedItem: { @@ -285,6 +291,15 @@ const styles = StyleSheet.create({ margin: 0, userSelect: 'none', whiteSpace: 'nowrap' + }, + + normalizedButton_navigationButton: { + display: 'inline-block', + width: '100%', + height: '100%', + + // cf: https://github.com/brave/browser-laptop/blob/b161b37cf5e9f59be64855ebbc5d04816bfc537b/less/navigationBar.less#L585 + padding: 0 } }) diff --git a/app/renderer/components/common/longPressButton.js b/app/renderer/components/common/longPressButton.js index 3b3d3df1a2f..710244b6ab3 100644 --- a/app/renderer/components/common/longPressButton.js +++ b/app/renderer/components/common/longPressButton.js @@ -4,6 +4,8 @@ const React = require('react') const ImmutableComponent = require('../immutableComponent') +const cx = require('../../../../js/lib/classSet') +const {StyleSheet, css} = require('aphrodite/no-important') class LongPressButton extends ImmutableComponent { constructor () { @@ -78,16 +80,42 @@ class LongPressButton extends ImmutableComponent { } render () { - return