Nulo/forgejo
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Fix some broken css (#23560) (#23567)

Browse source 
Backport #23560 by @wxiaoguang

1. The "close" inside "modal" are likely broken for long time
    * There is no var called `--body-color`
    * There is no `fullscreen modal`
* The `.ui.modal > .close.inside` doesn't seem to match most icons. It
only matches a few like "fork-repo-modal" or "adopt repo". Other places
are just buggy code copied again and again.
2. Convert the legacy `&:hover` LESS syntax to CSS syntax

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Co-authored-by: delvh <leon@kske.dev>
This commit is contained in:
Giteabot 2023-03-18 21:37:16 -04:00 committed by GitHub
parent f5a98b0f5b
commit 3f253b3f5a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
web_src/css/base.css
View file

@ -1173,10 +1173,8 @@ a.ui.card:hover,
border-color: var(--color-secondary);
}
.ui.modal > .close.inside,
.ui.fullscreen.modal > .close {
top: 11px; /* align modal close icon, for example admin notices */
color: var(--body-color);
.ui.modal > .close.inside {
color: var(--color-text);
}
.ui.basic.table > tbody > tr {

8
web_src/css/code/linebutton.css
View file

@ -17,8 +17,8 @@
left: 0;
transform: translateX(-70%);
cursor: pointer;
&:hover {
color: var(--color-primary);
}
}
.code-line-button:hover {
color: var(--color-primary);
}