mirror of
https://codeberg.org/forgejo/forgejo
synced 2025-10-19 13:00:51 +02:00
9a29241cde
Followup to https://codeberg.org/forgejo/forgejo/pulls/9317#issuecomment-7214470. Summary of changes: - implement new CSS for buttons that doesn't have problems Fomantic buttons have: - button height doesn't depend on it's content - `min-height:` is used instead of `padding-{top,bottom}:` - no margins to fit right in with the other elements while relying on `display:flex` and `flex-gap:` - other `.button` changes compared to `.ui.button`: - no semi-bold black text for now, I think it looks just as fine with normal weight - no variable `font-size` - should give better readability compared to existing `.small` and `.tiny` buttons - variable height depending on specified size (currently normal or `.small`) and on `@media (pointer: )` - apply the new buttons to areas where there are switches or dropdown openers near them Before: - https://codeberg.org/forgejo/forgejo/attachments/8d9a3941-8889-4420-8f4f-f44cb8c3726f - https://codeberg.org/forgejo/forgejo/attachments/e5804fae-a71f-43ba-918b-20d4f742124c - https://codeberg.org/forgejo/forgejo/attachments/65d4e31c-01b7-4050-89a1-4afe74aa574c - https://codeberg.org/forgejo/forgejo/attachments/c8057262-a834-4b61-a87f-70dab60b5506 - https://codeberg.org/forgejo/forgejo/attachments/aa9508e3-dca1-4ffe-913e-9cbddbb0d6ff - https://codeberg.org/forgejo/forgejo/attachments/4b2daa07-e0ca-4b44-8795-1609dff8968f - https://codeberg.org/forgejo/forgejo/attachments/93f7803f-1338-44dc-a428-e44e26231517 After: - https://codeberg.org/forgejo/forgejo/attachments/ca5426c4-d75e-493d-8b29-64eee1e1c9de - https://codeberg.org/forgejo/forgejo/attachments/f21c7219-6880-4a2e-9117-6267d46b3081 - https://codeberg.org/forgejo/forgejo/attachments/da3ee771-b5d5-4b4f-8c88-9a8de11b1a45 - https://codeberg.org/forgejo/forgejo/attachments/56f974c3-f72e-4f80-9bbd-5bc785a4a624 - https://codeberg.org/forgejo/forgejo/attachments/17f6e360-dfe5-4bb6-a6b5-dd747dbc2af8 - https://codeberg.org/forgejo/forgejo/attachments/91cd10e1-6c5e-4134-be4c-18223ed2a4f4 - https://codeberg.org/forgejo/forgejo/attachments/f0322c5c-ee9d-4889-9840-0c46838566cc - https://codeberg.org/forgejo/forgejo/attachments/96560fe7-3436-46dc-8456-c43bfaee2daf - https://codeberg.org/forgejo/forgejo/attachments/273cf795-6bb4-479f-804f-f40010fae825 Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9359 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org>
63 lines
1.7 KiB
CSS
63 lines
1.7 KiB
CSS
/* Copyright 2025 The Forgejo Authors. All rights reserved.
|
|
* SPDX-License-Identifier: GPL-3.0-or-later */
|
|
|
|
:root .switch {
|
|
/* The resulting switch height is --switch-item-min-height + 2px */
|
|
--switch-item-min-height: 34px;
|
|
--switch-padding-inline: 1.125em;
|
|
}
|
|
|
|
@media (pointer: coarse) {
|
|
:root .switch {
|
|
--switch-item-min-height: 38px;
|
|
}
|
|
}
|
|
|
|
.switch {
|
|
display: grid;
|
|
grid-auto-flow: column;
|
|
height: fit-content;
|
|
align-items: center;
|
|
align-self: center;
|
|
background: var(--color-menu);
|
|
border: 1px solid var(--color-input-border);
|
|
border-radius: var(--border-radius);
|
|
}
|
|
|
|
.switch > .item {
|
|
display: flex;
|
|
gap: 0.5rem;
|
|
align-items: center;
|
|
padding-inline: var(--switch-padding-inline);
|
|
min-height: var(--switch-item-min-height);
|
|
color: var(--color-text);
|
|
border-radius: var(--border-radius);
|
|
text-wrap: nowrap;
|
|
transition: background-color 0.1s ease;
|
|
}
|
|
|
|
.switch > .item:hover {
|
|
background: var(--color-hover);
|
|
}
|
|
|
|
/* Item that has to crawl under it's active neighbor, so when it is hovered,
|
|
there are no ugly unpainted v/^ shapes between them */
|
|
.switch > .item:has(+ .active.item) { /* Active neighbor is next item */
|
|
margin-right: calc(-1 * var(--border-radius));
|
|
padding-right: calc(var(--switch-padding-inline) + var(--border-radius));
|
|
}
|
|
.switch > .active.item + .item { /* Active neighbor is previous item */
|
|
margin-left: calc(-1 * var(--border-radius));
|
|
padding-left: calc(var(--switch-padding-inline) + var(--border-radius));
|
|
}
|
|
|
|
.switch > .active.item {
|
|
z-index: 2;
|
|
padding-left: var(--switch-padding-inline);
|
|
background: var(--color-active);
|
|
outline: 1px solid var(--color-input-border);
|
|
}
|
|
|
|
.switch > button.item {
|
|
background: transparent;
|
|
}
|