Skip to content

Feat: Add CM Qube 500 Case#2612

Merged
limetech merged 3 commits intounraid:masterfrom
Squidly271:Feat-Add-CM-Qube-500-case
Apr 22, 2026
Merged

Feat: Add CM Qube 500 Case#2612
limetech merged 3 commits intounraid:masterfrom
Squidly271:Feat-Add-CM-Qube-500-case

Conversation

@Squidly271
Copy link
Copy Markdown
Contributor

@Squidly271 Squidly271 commented Apr 13, 2026

Summary by CodeRabbit

  • Style
    • Added CM-Qube-500 case icon to the interface
    • Updated icon font assets

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 777a4388-0090-4ba5-8ad7-fbc82919eb5b

📥 Commits

Reviewing files that changed from the base of the PR and between c83718a and 3fc4c72.

⛔ Files ignored due to path filters (1)
  • emhttp/plugins/dynamix/styles/font-cases.woff is excluded by !**/*.woff
📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/styles/default-cases.css
🚧 Files skipped from review as they are similar to previous changes (1)
  • emhttp/plugins/dynamix/styles/default-cases.css

Walkthrough

A font-face source URL for the cases font was updated with a new version number, and a new CSS glyph mapping for the CM-Qube-500 case model icon was added.

Changes

Cohort / File(s) Summary
Font and Icon Glyph Updates
emhttp/plugins/dynamix/styles/default-cases.css
Updated @font-face src from version 20260209 to 20260413, and added new .case-CM-Qube-500:before glyph mapping with content \e94b.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A new case model hops into view,
With CM-Qube-500's icon so true,
The font refreshed to version new,
E-nine-four-bee shines through and through! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Feat: Add CM Qube 500 Case' directly describes the main change in the pull request, which adds support for a new case model.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 13, 2026

🔧 PR Test Plugin Available

A test plugin has been generated for this PR that includes the modified files.

Version: 2026.04.13.2121
Build: View Workflow Run

📥 Installation Instructions:

Install via Unraid Web UI:

  1. Go to Plugins → Install Plugin
  2. Copy and paste this URL:
https://preview.dl.unraid.net/pr-plugins/pr-2612/webgui-pr-2612.plg
  1. Click Install

Alternative: Direct Download

⚠️ Important Notes:

  • Testing only: This plugin is for testing PR changes
  • Backup included: Original files are automatically backed up
  • Easy removal: Files are restored when plugin is removed
  • Conflicts: Remove this plugin before installing production updates
  • Post-merge behavior: This preview stays available after merge until preview storage expires or it is manually cleaned up

📝 Modified Files:

Click to expand file list
emhttp/plugins/dynamix/styles/default-cases.css
emhttp/plugins/dynamix/styles/font-cases.woff

🔄 To Remove:

Navigate to Plugins → Installed Plugins and remove webgui-pr-2612, or run:

plugin remove webgui-pr-2612

🤖 This comment is automatically generated and will be updated with each new push to this PR.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@emhttp/plugins/dynamix/styles/default-cases.css`:
- Line 1: Update the pre-auth asset reference so the login page loads the new
font filename: change the reference to the cases font in emhttp/auth-request.php
from "/webGui/styles/font-cases.woff" to
"/webGui/styles/font-cases-20260413.woff" (or otherwise make both files
reference the same filename used in default-cases.css); search auth-request.php
for any other "/webGui/styles/font-cases" occurrences and update them to match
the new versioned name so pre-auth pages load the same font asset.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 56ed3314-e1ae-404b-a81d-de31f294da77

📥 Commits

Reviewing files that changed from the base of the PR and between 7756908 and 25b7942.

⛔ Files ignored due to path filters (1)
  • emhttp/plugins/dynamix/styles/font-cases-20260413.woff is excluded by !**/*.woff
📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/styles/default-cases.css

Comment thread emhttp/plugins/dynamix/styles/default-cases.css Outdated
@Squidly271 Squidly271 added 7.2 7.3 and removed 7.2 labels Apr 13, 2026
@limetech limetech merged commit bec8f0b into unraid:master Apr 22, 2026
2 checks passed
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.

2 participants