Skip to content

Simplifying Mobile User Experience - #4669

Merged
ildyria merged 2 commits into
masterfrom
disable-tiles-screen-size
Aug 25, 2026
Merged

Simplifying Mobile User Experience#4669
ildyria merged 2 commits into
masterfrom
disable-tiles-screen-size

Conversation

@ildyria

@ildyria ildyria commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Added a Sharing tile to the admin dashboard when uploads are enabled.
    • Added the Admin dashboard to Spotlight navigation.
    • Added responsive availability states for admin tools based on screen size.
  • UI Improvements
    • Improved maintenance duplicate-checker and row layouts with responsive content, actions, counts, and loading states.
    • Added clear small-screen messages when desktop layouts are unavailable.
    • Improved accessibility for disabled admin tiles with tooltips and appropriate keyboard behavior.
    • Excluded unavailable admin tools from Spotlight actions.

@ildyria
ildyria requested a review from a team as a code owner August 25, 2026 20:11
@ildyria ildyria added the v8 label Aug 25, 2026
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6d68f382-33d5-4c61-a066-f38843cb0d49

📥 Commits

Reviewing files that changed from the base of the PR and between beb38dd and 78704dc.

📒 Files selected for processing (1)
  • resources/js/v8/composables/spotlight/useSpotlightSystemActions.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

The PR adds responsive disabled states to admin tiles, includes the Admin dashboard in Spotlight results, adds small-screen messages to admin views, filters disabled tiles from Spotlight actions, and changes the maintenance duplicate checker to a responsive row layout.

Changes

Admin responsive navigation

Layer / File(s) Summary
Admin tile availability and disabled navigation
resources/js/v8/composables/useAdminTiles.ts, resources/js/v8/components/admin/AdminTileLink.vue
Admin tiles receive breakpoint-based disabled states. Disabled links block focus, clicks, and keyboard navigation. A sharing tile appears when uploads are permitted.
Admin view small-screen states
resources/js/v8/views/admin/Jobs.vue, resources/js/v8/views/admin/Purchasables.vue, resources/js/v8/views/admin/UserGroups.vue, resources/js/v8/views/admin/shop/PrintPixelSizesAdmin.vue
Admin management content is hidden below the medium breakpoint. The views show a localized small-screen message instead.
Spotlight Admin dashboard inclusion and filtering
resources/js/v8/composables/spotlight/useSpotlightNavItems.ts, resources/js/v8/composables/spotlight/useSpotlightSystemActions.ts
Spotlight results include the /admin route. Spotlight system actions exclude invisible or disabled admin tiles.

Maintenance duplicate-checker layout

Layer / File(s) Summary
Responsive duplicate-checker row
resources/js/v8/components/maintenance/MaintenanceRow.vue, resources/js/v8/components/maintenance/MaintenanceDuplicateChecker.vue
The duplicate checker uses MaintenanceRow. The row separates title, description, statistics, loading state, and actions across responsive layouts.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: 🔵 Low · up to 78704

The PR disables selected tiles for the mobile experience, but Spotlight navigation may still expose or navigate to those unavailable destinations. The change is mergeable with explicit owner awareness and follow-up to keep Spotlight navigation consistent with disabled tiles.

Poem

I’m a rabbit; admin tiles now know when to wait
Small screens show messages at the responsive gate
Spotlight keeps the dashboard in view
Disabled actions stay hidden there too
Maintenance rows align with care
Hop, hop, cleaner layouts everywhere

🚥 Pre-merge checks | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. Write docstrings for the functions missing them to satisfy the coverage threshold.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0925ef39-1993-48df-b782-bdcf4fdcd7e3

📥 Commits

Reviewing files that changed from the base of the PR and between 30b99fd and beb38dd.

📒 Files selected for processing (9)
  • resources/js/v8/components/admin/AdminTileLink.vue
  • resources/js/v8/components/maintenance/MaintenanceDuplicateChecker.vue
  • resources/js/v8/components/maintenance/MaintenanceRow.vue
  • resources/js/v8/composables/spotlight/useSpotlightNavItems.ts
  • resources/js/v8/composables/useAdminTiles.ts
  • resources/js/v8/views/admin/Jobs.vue
  • resources/js/v8/views/admin/Purchasables.vue
  • resources/js/v8/views/admin/UserGroups.vue
  • resources/js/v8/views/admin/shop/PrintPixelSizesAdmin.vue

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Comment thread resources/js/v8/composables/useAdminTiles.ts
@ildyria
ildyria merged commit 46cf04b into master Aug 25, 2026
22 checks passed
@ildyria
ildyria deleted the disable-tiles-screen-size branch August 25, 2026 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant