Skip to content

fix(settings/auth): prevent description tip from overlapping toggle column#757

Merged
Germey merged 1 commit into
mainfrom
fix/auth-tab-tip-overlap
May 16, 2026
Merged

fix(settings/auth): prevent description tip from overlapping toggle column#757
Germey merged 1 commit into
mainfrom
fix/auth-tab-tip-overlap

Conversation

@acedatacloud-dev
Copy link
Copy Markdown
Member

Summary

In the Settings → 登录方式 (Login Methods) tab the description text for 启用的登录方式 visually overlaps with the right-hand switch column (邮箱 / Google / GitHub / 手机号 / 微信). The tip text bleeds past the gutter into the area where the provider labels render — see the original report screenshot.

Root cause

user/Setting.vue defines the shared row layout via :deep(.settings-item):

:deep(.settings-item) { display: flex; gap: 16px; ... }
:deep(.settings-tip)  { max-width: 440px; ... }

.settings-label (which contains the tip) has no min-width: 0, so the tip's max-width: 440px resolves to the label's preferred flex size of ~440px. On the 50%-width dialog the actual content area for a row is only ~400–420px, so flex shrinking has to fight the label's intrinsic min-content. The right column .auth-providers-list (Auth.vue) was also pinned to min-width: 240px — wide enough that the two columns can't both fit, pushing the label content past the gutter and visually overlapping the toggles.

Fix

CSS-only and scoped to Auth.vue so other settings panels are untouched:

  • :deep(.settings-item) .settings-label { flex: 1 1 0; min-width: 0; } — let the label column grow into remaining row space and shrink below intrinsic min-content.
  • .auth-providers-content { flex: 0 0 auto; } — keep the right column at its content size so it can never push past the gutter.
  • .auth-providers-list { min-width: 220px; } (was 240px) — still plenty for the longest label (GitHub) plus the el-switch, but no longer wide enough to starve the tip.

Verification

  • No template / script / i18n changes.
  • Visually re-checked: at the 50% dialog width the description tip wraps cleanly inside its column and the switches sit in a separate gutter on the right.
  • Mobile fallback (@media (max-width: 640px) in user/Setting.vue) flips the row to a column anyway, so the new rules are a no-op on phones.
  • No interaction changes to the enabled toggles or the default_provider select.

This pull request was generated and committed by the GitHub Copilot coding agent on behalf of @CQUPTQiCu.

…olumn

The Login Methods (登录方式) tab renders each setting row as a flex
container whose .settings-label (title + tip) sits on the left and
.settings-content (the toggle list) on the right. The shared layout
in user/Setting.vue doesn't put min-width: 0 on
.settings-label though, so when its child .settings-tip carries
max-width: 440px the label's flex preferred size resolves to
~440px. On the 50%-width dialog there's roughly ~400-420px of content
area to share between the two columns, and the unlocked label squeezes
the toggle column out, visually overlapping the tip text
'勾选本站登录页面允许使用的登录方式。至少需要保留一种…' with the
provider labels (邮箱 / Google / GitHub / 手机号 / 微信).

Pin this panel's layout from inside Auth.vue (scoped) without touching
the shared layout other panels rely on:

* Let .settings-label grow into all remaining row space and
  shrink below intrinsic min-content (flex: 1 1 0; min-width: 0).
* Mark the right column as content-sized (flex: 0 0 auto) so it
  never gets pushed past the gutter.
* Reduce the toggle list's min-width from 240px to 220px — still
  enough for the longest provider label plus the el-switch, but no
  longer wide enough to starve the description tip on narrow dialogs.

No template / behaviour changes; this is a CSS-only fix scoped to
Auth.vue's existing <style lang="scss" scoped>.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 16, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
nexior 8390273 Commit Preview URL

Branch Preview URL
May 16 2026, 08:46 AM

@Germey Germey merged commit 9a1c236 into main May 16, 2026
2 checks passed
@Germey Germey deleted the fix/auth-tab-tip-overlap branch May 16, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants