Skip to content

Commit 5eea2aa

Browse files
committed
Use better icons for fullscreen toggle button
1 parent ea7398e commit 5eea2aa

4 files changed

Lines changed: 12 additions & 5 deletions

File tree

res/img/svg/maximise.svg

Lines changed: 1 addition & 0 deletions
Loading

res/img/svg/minimise.svg

Lines changed: 1 addition & 0 deletions
Loading

src/components/app/BottomBox.css

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@
7272

7373
.bottom-close-button,
7474
.bottom-assembly-button,
75-
.bottom-fullscreen-button {
75+
.bottom-fullscreen-hide-button,
76+
.bottom-fullscreen-show-button {
7677
width: 24px;
7778
height: 24px;
7879
flex-shrink: 0;
@@ -89,8 +90,12 @@
8990
background-image: url(firefox-profiler-res/img/svg/asm-icon.svg);
9091
}
9192

92-
.bottom-fullscreen-button {
93-
background-image: url(firefox-profiler-res/img/svg/open-in-new-12.svg);
93+
.bottom-fullscreen-show-button {
94+
background-image: url(firefox-profiler-res/img/svg/maximise.svg);
95+
}
96+
97+
.bottom-fullscreen-hide-button {
98+
background-image: url(firefox-profiler-res/img/svg/minimise.svg);
9499
}
95100

96101
.codeLoadingOverlay,

src/components/app/FullscreenToggleButton.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class FullscreenToggleButtonImpl extends React.PureComponent<Props> {
3535
<Localized id="BottomBox--hide-fullscreen" attrs={{ title: true }}>
3636
<button
3737
className={classNames(
38-
'bottom-fullscreen-button',
38+
'bottom-fullscreen-hide-button',
3939
'photon-button',
4040
'photon-button-ghost',
4141
'photon-button-ghost--checked'
@@ -49,7 +49,7 @@ class FullscreenToggleButtonImpl extends React.PureComponent<Props> {
4949
<Localized id="BottomBox--show-fullscreen" attrs={{ title: true }}>
5050
<button
5151
className={classNames(
52-
'bottom-fullscreen-button',
52+
'bottom-fullscreen-show-button',
5353
'photon-button',
5454
'photon-button-ghost'
5555
)}

0 commit comments

Comments
 (0)