fix: improve mobile admin layouts - #107
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 576cbe8642
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| </Tooltip> | ||
| <Tooltip title={credential.filename}> | ||
| <Text ellipsis type="secondary"> | ||
| <Text className="account-status-card-filename" type="secondary"> |
There was a problem hiding this comment.
Retain filename containment above 767px
At viewport widths of 768px or more, none of the new .account-status-card-* overflow rules apply, but this line removes ellipsis unconditionally. Because /admin-api/credentials accepts caller-supplied basename-only filenames without a length limit, a long unbroken filename can retain its intrinsic width on tablet or desktop and overflow the card or displace the Refresh button; keep truncation outside the mobile breakpoint or apply the min-width/wrapping constraints at every width.
Useful? React with 👍 / 👎.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #107 +/- ##
=======================================
Coverage 94.77% 94.77%
=======================================
Files 21 21
Lines 4251 4251
Branches 1238 1238
=======================================
Hits 4029 4029
Misses 222 222
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Summary
Verification