Skip to content

Commit c8b778a

Browse files
committed
improve gap variables
1 parent 43e7745 commit c8b778a

File tree

4 files changed

+9
-7
lines changed

4 files changed

+9
-7
lines changed

web_src/css/base.css

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,9 @@
3636
--z-index-toast: 1002; /* should be larger than modal */
3737

3838
--font-size-label: 12px; /* font size of individual labels */
39-
--gap-label: .25rem; /* gap between labels */
39+
40+
--gap-inline: 0.25rem; /* gap for inline texts and elements, for example: the spaces for sentence with labels, button text, etc */
41+
--gap-block: 0.25rem; /* gap for element blocks, for example: spaces between buttons, menu image & title, header icon & title etc */
4042
}
4143

4244
@media (min-width: 768px) and (max-width: 1200px) {
@@ -1096,7 +1098,7 @@ table th[data-sortt-desc] .svg {
10961098
.flex-text-inline {
10971099
display: inline-flex;
10981100
align-items: center;
1099-
gap: .25rem;
1101+
gap: var(--gap-inline);
11001102
vertical-align: middle;
11011103
min-width: 0; /* make ellipsis work */
11021104
}
@@ -1124,7 +1126,7 @@ table th[data-sortt-desc] .svg {
11241126
.flex-text-block {
11251127
display: flex;
11261128
align-items: center;
1127-
gap: .5rem;
1129+
gap: var(--gap-block);
11281130
min-width: 0;
11291131
}
11301132

@@ -1139,7 +1141,7 @@ the "!important" is necessary to override Fomantic UI menu item styles, meanwhil
11391141
.ui.dropdown .menu.flex-items-menu > .item:not(.hidden, .filtered, .tw-hidden) {
11401142
display: flex !important;
11411143
align-items: center;
1142-
gap: .5rem;
1144+
gap: var(--gap-block);
11431145
min-width: 0;
11441146
}
11451147
.ui.dropdown .menu.flex-items-menu > .item img,

web_src/css/modules/label.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
.ui.label {
55
display: inline-flex;
66
align-items: center;
7-
gap: var(--gap-label);
7+
gap: var(--gap-inline);
88
min-width: 0;
99
max-width: 100%;
1010
background: var(--color-label-bg);

web_src/css/repo.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@
8484

8585
.issue-content-right .ui.list.labels-list {
8686
display: flex;
87-
gap: var(--gap-label);
87+
gap: var(--gap-inline);
8888
flex-wrap: wrap;
8989
}
9090

web_src/css/shared/flex-list.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
align-items: center;
4949
/* labels are under effect of this gap here because they are display:contents. Ideally we should make wrapping
5050
of labels work without display: contents and set this to a static value again. */
51-
gap: var(--gap-label);
51+
gap: var(--gap-inline);
5252
max-width: 100%;
5353
color: var(--color-text);
5454
font-size: 16px;

0 commit comments

Comments
 (0)