Repository navigation
fix(dashboard): make API Manager key table usable on mobile - #4556
Merged
diegosouzapw merged 3 commits intoJun 21, 2026
Merged
diegosouzapw merged 3 commits into
diegosouzapw merged 3 commits into
Conversation
The registered API keys table on /dashboard/api-manager was unusable on small screens: the 12-column grid collapsed and squeezed content, action icons (view costs, regenerate, permissions, delete) were hidden behind hover which doesn't exist on touch, and there was no horizontal scroll. - Wrap both key tables (normal + quota) in overflow-x-auto so they can be scrolled sideways instead of collapsing. - Add min-w-[760px] to header + row so columns keep their intended widths and the scroll is meaningful. - Switch all hover-gated icons (cost/regenerate/permissions/delete + the 'key only available at creation' info lock) to opacity-100 on mobile and keep the hover-reveal behavior on >=sm screens. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
…->2983 Mobile-responsive table fix adds a horizontal-scroll wrapper + min-w + mobile-visible action buttons (+4 lines). Baseline bumped to match (wc+1). Fast-gate PR->release does not run check:file-size; reconciled in-branch per Hard Rule cheaper-inference#9. Co-authored-by: diegosouzapw <diegosouza.pw@gmail.com>
Collaborator
|
Thanks @janeza2! The horizontal-scroll wrapper + |
diegosouzapw
merged commit Jun 21, 2026
9447de0
into
cheaper-inference:release/v3.8.33
3 checks passed
Merged
tkgo11
pushed a commit
to tkgo11/OmniRoute
that referenced
this pull request
Sep 23, 2026
…inference#4556) Integrated into release/v3.8.33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The registered API keys table on
/dashboard/api-managerwas unusable on small screens:hover, which does not exist on touch devicesChanges
ApiManagerPageClient.tsxoverflow-x-autoso they can be scrolled sideways instead of collapsing. Addmin-w-[760px]to header + row so columns keep their intended widths and the scroll is meaningful. Switch all hover-gated icons (cost / regenerate / permissions / delete + the "key only available at creation" info lock) toopacity-100 sm:opacity-0 sm:group-hover:opacity-100, so they are always visible on mobile (<sm) while keeping the desktop hover-reveal behavior.Behavior
Verification
api-manager-page-static,api-manager-quota-keys-section,api-manager-filters, etc.)anybudget unchanged (noanyintroduced)sm:prefixTest plan
/dashboard/api-manageron a narrow viewport (<640px) — confirm the table now scrolls horizontally instead of collapsing, and that view-costs / regenerate / permissions / delete icons are visible without hovering.