Skip to content

Commit 6824d66

Browse files
authored
Fix the height of the buttons in the ribbon (#1843)
* Make the height of the first buttons the same as the rest of the ribbon list * Make everything the same height
1 parent 1a353f0 commit 6824d66

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

webview/src/plots/components/ribbon/Ribbon.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,14 @@ export const Ribbon: React.FC<RibbonProps> = ({ revisions }) => {
3535

3636
return (
3737
<ul className={styles.list} data-testid="ribbon">
38-
<li className={styles.addButtonWrapper}>
38+
<li className={styles.buttonWrapper}>
3939
<IconButton
4040
onClick={selectRevisions}
4141
icon={AllIcons.LINES}
4242
text={`${revisions.length} of ${MAX_NB_EXP}`}
4343
/>
4444
</li>
45-
<li className={styles.addButtonWrapper}>
45+
<li className={styles.buttonWrapper}>
4646
<IconButton
4747
onClick={refreshRevisions}
4848
icon={AllIcons.REFRESH}

webview/src/plots/components/ribbon/styles.module.scss

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
.block {
1111
border-left-style: solid;
1212
border-left-width: 3px;
13-
padding: 4px;
1413
background-color: $header-bg-color;
1514
color: $fg-color;
1615
display: inline-flex;
@@ -31,7 +30,8 @@
3130
}
3231

3332
.label {
34-
height: 30px;
33+
height: 26px;
34+
padding: 2px 4px;
3535
background-color: transparent;
3636
border: none;
3737
text-align: left;
@@ -40,17 +40,20 @@
4040
flex-direction: column;
4141
justify-content: center;
4242
min-width: max-content;
43+
line-height: 1.1;
4344
}
4445

4546
.title {
4647
display: inline-flex;
4748
align-items: center;
4849
padding-right: 15px;
4950
position: relative;
51+
font-size: 0.8rem;
5052
}
5153

5254
.subtitle {
5355
color: $watermark-color;
56+
font-size: 0.5625rem;
5457
}
5558

5659
.clearButton {
@@ -78,7 +81,6 @@
7881
flex-wrap: wrap;
7982
}
8083

81-
.addButtonWrapper {
82-
background-color: $accent-color;
84+
.buttonWrapper {
8385
list-style: none;
8486
}

0 commit comments

Comments
 (0)