Skip to content

Add HL-15 to case font#2534

Merged
limetech merged 2 commits intounraid:masterfrom
Squidly271:patch-6
Feb 4, 2026
Merged

Add HL-15 to case font#2534
limetech merged 2 commits intounraid:masterfrom
Squidly271:patch-6

Conversation

@Squidly271
Copy link
Contributor

@Squidly271 Squidly271 commented Jan 24, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Style
    • Updated font assets to the latest version for improved consistency
    • Extended case identifier styling to support an additional hardware configuration

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions
Copy link

github-actions bot commented Jan 24, 2026

🔧 PR Test Plugin Available

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

Version: 2026.01.24.0249
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-2534/webgui-pr-2534.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

📝 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-2534, or run:

plugin remove webgui-pr-2534

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 24, 2026 •
edited
Loading

Walkthrough

A CSS stylesheet update that refreshes the font asset version parameter and introduces a new icon glyph class for the 45Drives HL15 case identifier.

Changes

Cohort / File(s) Summary
Font and Icon Updates
emhttp/plugins/dynamix/styles/default-cases.css		 Updated font-face query parameter from v=20240608 to v=20260123; added .case-45Drives-HL15:before CSS rule with glyph content '\e94a'

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A font refreshed, a version new,
And icons gleaming in digital hue,
Forty-five drives find their place,
With glyphs that grace each case! ✨

🚥 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 'Add HL-15 to case font' directly and accurately describes the main change in the pull request: adding a new CSS rule for the HL-15 case identifier to the cases font file.
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.

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • HL-15: Entity not found: Issue - Could not find referenced Issue.

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

@Squidly271 Squidly271 added the 7.3 label Jan 27, 2026
@limetech limetech merged commit a2d4af1 into unraid:master Feb 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

7.3

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Squidly271 @limetech