Skip to content

Commit ea54b3f

Browse files
author
Frederic Collonval
committed
Use more IIconRegistry
1 parent 7169e9f commit ea54b3f

File tree

8 files changed

+46
-48
lines changed

8 files changed

+46
-48
lines changed

src/components/SinglePastCommitInfo.tsx

Lines changed: 25 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
import { JupyterFrontEnd } from '@jupyterlab/application';
2+
import { IRenderMimeRegistry } from '@jupyterlab/rendermime';
3+
import { defaultIconRegistry } from '@jupyterlab/ui-components';
24
import * as React from 'react';
35
import { classes } from 'typestyle/';
46
import { fileIconStyle } from '../componentsStyle/FileItemStyle';
@@ -13,25 +15,21 @@ import {
1315
commitDetailStyle,
1416
commitOverviewNumbers,
1517
commitStyle,
16-
deletionIconStyle,
1718
diffIconStyle,
1819
floatRightStyle,
1920
iconStyle,
20-
insertionIconStyle,
21-
numberofChangedFilesStyle,
2221
revertButtonStyle
2322
} from '../componentsStyle/SinglePastCommitInfoStyle';
2423
import {
25-
ICommitModifiedFile,
2624
Git,
27-
ISingleCommitInfo,
28-
IDiffCallback
25+
ICommitModifiedFile,
26+
IDiffCallback,
27+
ISingleCommitInfo
2928
} from '../git';
29+
import { isDiffSupported } from './diff/Diff';
30+
import { openDiffView } from './diff/DiffWidget';
3031
import { parseFileExtension } from './FileList';
3132
import { ResetDeleteSingleCommit } from './ResetDeleteSingleCommit';
32-
import { openDiffView } from './diff/DiffWidget';
33-
import { IRenderMimeRegistry } from '@jupyterlab/rendermime';
34-
import { isDiffSupported } from './diff/Diff';
3533

3634
export interface ISinglePastCommitInfoProps {
3735
topRepoPath: string;
@@ -141,15 +139,30 @@ export class SinglePastCommitInfo extends React.Component<
141139
<div className={commitStyle}>
142140
<div className={commitOverviewNumbers}>
143141
<span>
144-
<span className={classes(iconStyle, numberofChangedFilesStyle)} />
142+
{defaultIconRegistry.iconReact({
143+
name: 'file',
144+
className: classes(iconStyle),
145+
tag: 'span',
146+
title: '# Files Changed'
147+
})}
145148
{this.state.filesChanged}
146149
</span>
147150
<span>
148-
<span className={classes(iconStyle, insertionIconStyle)} />
151+
{defaultIconRegistry.iconReact({
152+
name: 'git-insertionsMade',
153+
className: classes(iconStyle),
154+
tag: 'span',
155+
title: '# Insertions'
156+
})}
149157
{this.state.insertionCount}
150158
</span>
151159
<span>
152-
<span className={classes(iconStyle, deletionIconStyle)} />
160+
{defaultIconRegistry.iconReact({
161+
name: 'git-deletionsMade',
162+
className: classes(iconStyle),
163+
tag: 'span',
164+
title: '# Deletions'
165+
})}
153166
{this.state.deletionCount}
154167
</span>
155168
</div>

src/componentsStyle/SinglePastCommitInfoStyle.tsx

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -67,26 +67,11 @@ export const commitDetailFilePathStyle = style({
6767

6868
export const iconStyle = style({
6969
display: 'inline-block',
70-
width: '20px',
70+
width: '13px',
7171
height: '13px',
72-
backgroundPosition: 'center',
73-
backgroundRepeat: 'no-repeat',
74-
backgroundSize: '14px',
7572
right: '10px'
7673
});
7774

78-
export const numberofChangedFilesStyle = style({
79-
backgroundImage: 'var(--jp-icon-file)'
80-
});
81-
82-
export const insertionIconStyle = style({
83-
backgroundImage: 'var(--jp-icon-insertions-made)'
84-
});
85-
86-
export const deletionIconStyle = style({
87-
backgroundImage: 'var(--jp-icon-deletions-made)'
88-
});
89-
9075
export const diffIconStyle = style({
9176
backgroundColor: 'transparent',
9277
backgroundPosition: 'center',

src/icons.ts

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,22 @@ import { IIconRegistry } from '@jupyterlab/ui-components';
22

33
// icon svg import statements
44
import gitSvg from '../style/images/git-icon.svg';
5+
import deletionsMadeSvg from '../style/images/deletions-made-icon.svg';
6+
import insertionsMadeSvg from '../style/images/insertions-made-icon.svg';
57

68
export function registerGitIcons(iconRegistry: IIconRegistry) {
7-
iconRegistry.addIcon({
8-
name: 'git',
9-
svg: gitSvg
10-
});
9+
iconRegistry.addIcon(
10+
{
11+
name: 'git',
12+
svg: gitSvg
13+
},
14+
{
15+
name: 'git-deletionsMade',
16+
svg: deletionsMadeSvg
17+
},
18+
{
19+
name: 'git-insertionsMade',
20+
svg: insertionsMadeSvg
21+
}
22+
);
1123
}

style/images/deletions-made-icon-white.svg

Lines changed: 0 additions & 4 deletions
This file was deleted.

style/images/deletions-made-icon.svg

Lines changed: 2 additions & 2 deletions
Loading

style/images/insertions-made-icon-white.svg

Lines changed: 0 additions & 4 deletions
This file was deleted.

style/images/insertions-made-icon.svg

Lines changed: 2 additions & 2 deletions
Loading

style/variables.css

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,9 @@
2727

2828
[data-jp-theme-light='true'] {
2929
--jp-icon-arrow-down: url('./images/arrow-down.svg');
30-
--jp-icon-deletions-made: url('./images/deletions-made-icon.svg');
3130
--jp-icon-discard-file: url('./images/discard.svg');
3231
--jp-icon-git-pull: url('./images/git-pull.svg');
3332
--jp-icon-git-push: url('./images/git-push.svg');
34-
--jp-icon-insertions-made: url('./images/insertions-made-icon.svg');
3533
--jp-icon-move-file-down: url('./images/move-file-down.svg');
3634
--jp-icon-move-file-up: url('./images/move-file-up.svg');
3735
--jp-icon-plus: url('./images/plus.svg');
@@ -41,11 +39,9 @@
4139

4240
[data-jp-theme-light='false'] {
4341
--jp-icon-arrow-down: url('./images/arrow-down-white.svg');
44-
--jp-icon-deletions-made: url('./images/deletions-made-icon-white.svg');
4542
--jp-icon-discard-file: url('./images/discard-selected.svg');
4643
--jp-icon-git-pull: url('./images/git-pull-white.svg');
4744
--jp-icon-git-push: url('./images/git-push-white.svg');
48-
--jp-icon-insertions-made: url('./images/insertions-made-icon-white.svg');
4945
--jp-icon-move-file-down-white: url('./images/move-file-down-hover.svg');
5046
--jp-icon-move-file-up: url('./images/move-file-up-hover.svg');
5147
--jp-icon-plus: url('./images/plus-white.svg');

0 commit comments

Comments
 (0)