Skip to content

Commit

Permalink
Revert overflow: overlay (revert go-gitea#21850) (go-gitea#25231) (go…
Browse files Browse the repository at this point in the history
…-gitea#25239)

Backport go-gitea#25231 by @wxiaoguang

It causes not only one issue like go-gitea#25221 (the footer width was also
affected by that change and was fixed some time ago)

The problem of "overflow: overlay" (go-gitea#21850) is:

* It's not widely supported and is non-standard
https://caniuse.com/css-overflow-overlay
* It's not widely tested in Gitea (some standard layout like `ui
container + ui grid` may break it).
* The benefit seems smaller than the problems it brings.

So, I think it is good to revert it.

----

Let's leave enough time for testing and reviewing.

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Co-authored-by: silverwind <me@silverwind.io>
  • Loading branch information
3 people authored Jun 13, 2023
1 parent fa28d0e commit 2294804
Show file tree
Hide file tree
Showing 5 changed files with 7,345 additions and 16 deletions.
17 changes: 3 additions & 14 deletions web_src/css/base.css
Original file line number Diff line number Diff line change
Expand Up @@ -266,19 +266,11 @@ body {
color: var(--color-text);
background-color: var(--color-body);
tab-size: var(--tab-size);
overflow-y: auto;
display: flex;
flex-direction: column;
overflow-wrap: break-word;
}

@supports (overflow: overlay) {
body {
overflow: overlay; /* stylelint-disable-line */
scrollbar-gutter: stable;
}
}

table {
border-collapse: collapse;
}
Expand Down Expand Up @@ -910,7 +902,7 @@ img.ui.avatar,
/* overwrite semantic width of containers inside the main page content div (div with class "page-content") */
.page-content .ui.ui.ui.container:not(.fluid) {
width: 1280px;
max-width: calc(100vw - 64px);
max-width: calc(100% - 64px);
margin-left: auto;
margin-right: auto;
}
Expand All @@ -922,8 +914,7 @@ img.ui.avatar,
/* enable fluid page widths for medium size viewports */
@media (min-width: 768px) and (max-width: 1200px) {
.page-content .ui.ui.ui.container:not(.fluid) {
width: calc(100vw - 32px);
max-width: calc(100vw - 32px);
max-width: calc(100% - 32px);
}
.ui.container.fluid.padded {
padding: 0 16px;
Expand All @@ -932,8 +923,7 @@ img.ui.avatar,

@media (max-width: 767.98px) {
.page-content .ui.ui.ui.container:not(.fluid) {
width: calc(100vw - 16px);
max-width: calc(100vw - 16px);
max-width: calc(100% - 16px);
}
.ui.container.fluid.padded {
padding: 0 8px;
Expand All @@ -945,7 +935,6 @@ img.ui.avatar,
align-items: center;
background: var(--color-nav-bg);
border-bottom: 1px solid var(--color-secondary);
width: 100vw;
min-height: 52px;
margin: 0 !important;
}
Expand Down
4 changes: 3 additions & 1 deletion web_src/fomantic/.npmrc
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
audit=false
fund=false
update-notifier=false
package-lock=false
package-lock=true
save-exact=true
lockfile-version=3
optional=false
1 change: 1 addition & 0 deletions web_src/fomantic/_site/globals/site.variables
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,4 @@
@variationPopupTooltip: false;
@linkHoverUnderline: underline;
@variationButtonSocial: false;
@pageOverflowX: visible;
2 changes: 1 addition & 1 deletion web_src/fomantic/build/semantic.css

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 2294804

Please sign in to comment.