fix: polish subscription limits interactions - #1
Conversation
|
Ran the full dashboard suite locally to verify this PR — great UI polish! One follow-up found: I've pushed a minimal test-only adaptation as a stacked PR into your branch: LceAn#1 ( Feel free to merge it, cherry-pick, or rework as you prefer 🙏 本地跑全量验证了这个 PR,UI 打磨很棒!一个跟进点:未被本 PR 改动的 我已把仅测试的最小适配以叠加 PR 推到你分支:LceAn#1。用的就是你在 直接合并、cherry-pick 或按你的思路改都欢迎 🙏 |
|
Hi @LceAn, thanks again for this PR — it's been merged into The subscription settings card redesign is the clear highlight — the wider two-column form and the scrollable popover feel much more comfortable to use. Really nice work there! All of it is fully retained in follow-up commit After reviewing everything against the live product, we did revert the 你好 @LceAn,再次感谢这个 PR——已合并进 订阅管理弹窗的改版是这次最大的亮点——加宽的双列表单和可滚动的弹层用起来舒服多了,做得真好!这部分在后续提交 实机对比后我们将 |
This is a focused follow-up to xiufengsun#464, based on
Hu9956:feat/subscription-managerataa0b0b22.What changed
role="button"wrapper with a real header button and a separate content region. This prevents edit/delete/copy controls inside a provider row from accidentally toggling the row.aria-controls, animated expand affordance, selected-row surface treatment, and more integrated subscription progress/detail styling.No storage or API behavior is changed.
Validation
validate:copy,validate:locale,validate:ui-hardcode,validate:guardrails: passedThe full root test run still contains an unrelated existing leaderboard refresh failure on this Windows checkout; the subscription-manager targeted tests pass except for the pre-existing Windows permission test that cannot simulate unreadable files.