Skip to content

Commit 8e0b86a

Browse files
author
0ko
committed
fix(ui): overflow tabular menu CSS fixes (go-gitea#7578)
Change 1 Followup to https://codeberg.org/forgejo/forgejo/pulls/7314 and https://codeberg.org/forgejo/forgejo/pulls/7356 Replace baseline with center, because baseline alignment is broken with Asian scripts. Preview: https://codeberg.org/attachments/f612ca9b-5033-44f1-8bea-b71170c1f595 - bug https://codeberg.org/attachments/69e021eb-96dc-40b1-8c65-9fb4a378e051 - with fix applied Change 2 Followup to https://codeberg.org/forgejo/forgejo/pulls/7108 and https://codeberg.org/forgejo/forgejo/issues/4169 Rescope `.ui.tabular.menu .item { height: 100%;}` to overflow menu, because it was only meant for overflow menu based on PR context, but started to negatively impact the other "bookmark" style menus: they've got an extra bottom line under active tabs. Preview: https://codeberg.org/attachments/b535e437-16fd-4af2-ae8c-221618ec6b8c - bug https://codeberg.org/attachments/10a3086d-8fe7-4317-bd66-71e8139b2cc7 - bug in another area https://codeberg.org/attachments/6a6b8994-c982-48d0-98fe-68e70ba709c2 - with fix applied https://codeberg.org/attachments/8fd8a332-a9ec-4b28-948e-a4e986e301da - intended fix still applies Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/7578 Reviewed-by: Michael Kriese <[email protected]> Reviewed-by: Gusted <[email protected]>
1 parent 424606c commit 8e0b86a

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

web_src/css/base.css

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1054,10 +1054,11 @@ overflow-menu .overflow-menu-items {
10541054

10551055
overflow-menu .overflow-menu-items .item {
10561056
margin-bottom: 0 !important; /* reset fomantic's margin, because the active menu has special bottom border */
1057+
height: 100%;
10571058
}
10581059

10591060
overflow-menu .overflow-menu-items .item > .svg {
1060-
align-self: baseline;
1061+
align-self: center;
10611062
}
10621063

10631064
overflow-menu .ui.label {
@@ -1360,10 +1361,6 @@ table th[data-sortt-desc] .svg {
13601361
border-color: var(--color-secondary);
13611362
}
13621363

1363-
.ui.tabular.menu .item {
1364-
height: 100%;
1365-
}
1366-
13671364
.ui.tabular.menu .item,
13681365
.ui.secondary.pointing.menu .item {
13691366
padding: 11px 12px !important;

0 commit comments

Comments
 (0)