Skip to content

Commit 0722c25

Browse files
committed
fixed requested changes
1 parent 45e75cd commit 0722c25

File tree

4 files changed

+9
-11
lines changed

4 files changed

+9
-11
lines changed

browser/components/CodeEditor.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ export default class CodeEditor extends React.Component {
295295
inputStyle: 'textarea',
296296
dragDrop: false,
297297
direction: RTL ? 'rtl' : 'ltr',
298-
rtlMoveVisually: RTL ? 'true' : 'false',
298+
rtlMoveVisually: RTL,
299299
foldGutter: true,
300300
lint: enableMarkdownLint ? this.getCodeEditorLintConfig() : false,
301301
gutters: ['CodeMirror-linenumbers', 'CodeMirror-foldgutter', 'CodeMirror-lint-markers'],
@@ -559,7 +559,7 @@ export default class CodeEditor extends React.Component {
559559
}
560560
if (prevProps.RTL !== this.props.RTL) {
561561
this.editor.setOption('direction', this.props.RTL ? 'rtl' : 'ltr')
562-
this.editor.setOption('rtlMoveVisually', this.props.RTL ? 'true' : 'false')
562+
this.editor.setOption('rtlMoveVisually', this.props.RTL)
563563
}
564564
if (prevProps.enableMarkdownLint !== enableMarkdownLint || prevProps.customMarkdownLintConfig !== customMarkdownLintConfig) {
565565
if (!enableMarkdownLint) {

browser/components/MarkdownPreview.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,6 @@ body {
107107
${optimizeOverflowScroll ? 'height: 100%;' : ''}
108108
${RTL ? 'direction: rtl;' : ''}
109109
${RTL ? 'text-align: right;' : ''}
110-
111-
112110
}
113111
@media print {
114112
body {

browser/main/Detail/MarkdownNoteDetail.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -484,7 +484,7 @@ class MarkdownNoteDetail extends React.Component {
484484
</div>
485485
<div styleName='info-right'>
486486
<ToggleModeButton onClick={(e) => this.handleSwitchMode(e)} editorType={editorType} />
487-
<ToggleDirectionButton onClick={(e) => this.handleSwitchDirection(e)} editorDirection={this.state.RTL} />
487+
<ToggleDirectionButton onClick={(e) => this.handleSwitchDirection(e)} isRTL={this.state.RTL} />
488488
<StarButton
489489
onClick={(e) => this.handleStarButtonClick(e)}
490490
isActive={note.isStarred}

browser/main/Detail/ToggleDirectionButton.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,22 @@ import styles from './ToggleDirectionButton.styl'
55
import i18n from 'browser/lib/i18n'
66

77
const ToggleDirectionButton = ({
8-
onClick, editorDirection
8+
onClick, isRTL
99
}) => (
1010
<div styleName='control-toggleModeButton'>
11-
<div styleName={editorDirection ? 'active' : undefined} onClick={() => onClick()}>
12-
<img src={!editorDirection ? '../resources/icon/icon-left-to-right.svg' : ''} />
11+
<div styleName={isRTL ? 'active' : undefined} onClick={() => onClick()}>
12+
<img src={!isRTL ? '../resources/icon/icon-left-to-right.svg' : ''} />
1313
</div>
14-
<div styleName={!editorDirection ? 'active' : undefined} onClick={() => onClick()}>
15-
<img src={!editorDirection ? '' : '../resources/icon/icon-right-to-left.svg'} />
14+
<div styleName={!isRTL ? 'active' : undefined} onClick={() => onClick()}>
15+
<img src={!isRTL ? '' : '../resources/icon/icon-right-to-left.svg'} />
1616
</div>
1717
<span lang={i18n.locale} styleName='tooltip'>{i18n.__('Toggle Direction')}</span>
1818
</div>
1919
)
2020

2121
ToggleDirectionButton.propTypes = {
2222
onClick: PropTypes.func.isRequired,
23-
editorDirection: PropTypes.string.isRequired
23+
isRTL: PropTypes.string.isRequired
2424
}
2525

2626
export default CSSModules(ToggleDirectionButton, styles)

0 commit comments

Comments
 (0)