fix source buttons not displaying for file renaming

This commit is contained in:
Konstantin Schaper
2020-08-11 10:09:28 +02:00
parent 94bac2ad37
commit c9ccaed61d
2 changed files with 19 additions and 19 deletions

View File

@@ -415,25 +415,25 @@ class DiffFile extends React.Component<Props, State> {
} }
const collapseIcon = this.hasContent(file) ? <Icon name={icon} color="inherit"/> : null; const collapseIcon = this.hasContent(file) ? <Icon name={icon} color="inherit"/> : null;
const fileControls = fileControlFactory ? fileControlFactory(file, this.setCollapse) : null; const fileControls = fileControlFactory ? fileControlFactory(file, this.setCollapse) : null;
const sideBySideToggle = const sideBySideToggle = file.hunks && file.hunks.length && <MenuContext.Consumer>
file.hunks && file.hunks.length > 0 && ( {({setCollapsed}) => (
<DiffButton
icon={sideBySide ? "align-left" : "columns"}
tooltip={t(sideBySide ? "diff.combined" : "diff.sideBySide")}
onClick={() =>
this.toggleSideBySide(() => {
if (this.state.sideBySide) {
setCollapsed(true);
}
})
}
/>
)}
</MenuContext.Consumer>;
const headerButtons = (
<ButtonWrapper className={classNames("level-right", "is-flex")}> <ButtonWrapper className={classNames("level-right", "is-flex")}>
<ButtonGroup> <ButtonGroup>
<MenuContext.Consumer> {sideBySideToggle}
{({setCollapsed}) => (
<DiffButton
icon={sideBySide ? "align-left" : "columns"}
tooltip={t(sideBySide ? "diff.combined" : "diff.sideBySide")}
onClick={() =>
this.toggleSideBySide(() => {
if (this.state.sideBySide) {
setCollapsed(true);
}
})
}
/>
)}
</MenuContext.Consumer>
{fileControls} {fileControls}
</ButtonGroup> </ButtonGroup>
</ButtonWrapper> </ButtonWrapper>
@@ -467,7 +467,7 @@ class DiffFile extends React.Component<Props, State> {
</TitleWrapper> </TitleWrapper>
{this.renderChangeTag(file)} {this.renderChangeTag(file)}
</FullWidthTitleHeader> </FullWidthTitleHeader>
{sideBySideToggle} {headerButtons}
</FlexWrapLevel> </FlexWrapLevel>
</div> </div>
{body} {body}

View File

@@ -177,7 +177,7 @@ class RepositoryRoot extends React.Component<Props> {
break; break;
default: default:
if (targetLink) { if (targetLink) {
links.push(sourceLink, targetLink); links.push(targetLink, sourceLink); // Target link first because its the previous file
} else { } else {
links.push(sourceLink); links.push(sourceLink);
} }