Skip to content

Fix: Create snapshot display aberrations#2420

Merged
limetech merged 1 commit into
unraid:masterfrom
Squidly271:Fix/VMSnapshot
Oct 15, 2025
Merged

Fix: Create snapshot display aberrations#2420
limetech merged 1 commit into
unraid:masterfrom
Squidly271:Fix/VMSnapshot

Conversation

@Squidly271

@Squidly271 Squidly271 commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Style
    • Refined VM Manager form layout by setting the “Target snapshot” text field to 60% width, improving readability and alignment with surrounding elements.
    • Enhances visual consistency across form inputs and reduces crowding on narrower screens.
    • No functional changes; this is a visual/UI polish aimed at a cleaner, more balanced appearance.

@coderabbitai

coderabbitai Bot commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Adds a single CSS rule in VMMachines.css to set the width of the text input with id "targetsnap" to 60%. No other selectors or layout rules are modified.

Changes

Cohort / File(s) Summary
VM Manager Styles
emhttp/plugins/dynamix.vm.manager/sheets/VMMachines.css
Added CSS: input[type="text"]#targetsnap { width: 60%; }. No other changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nibble CSS like garden greens,
A tidy tweak between the scenes—
Sixty percent for snaps to grow,
Inputs snug in UI’s flow.
Thump-thump! I hop, commit complete,
Small style change, sleek and neat. 🐇✨

Pre-merge checks and finishing touches

✅ 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 directly references fixing display aberrations in the create snapshot interface, which matches the main change of adding a CSS rule to correct the input field width and resolve visual glitches; while the phrasing is slightly unconventional, it still clearly relates to the core update.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 95bc896 and 8a27d06.

📒 Files selected for processing (1)
  • emhttp/plugins/dynamix.vm.manager/sheets/VMMachines.css (1 hunks)

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

@github-actions

github-actions Bot commented Oct 3, 2025

Copy link
Copy Markdown

🔧 PR Test Plugin Available

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

Version: 2025.10.03.1948
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-2420/webgui-pr-2420.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.vm.manager/sheets/VMMachines.css

🔄 To Remove:

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

plugin remove webgui-pr-2420

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

@ljm42 ljm42 added the 7.2 label Oct 13, 2025
@limetech limetech merged commit 9e70f88 into unraid:master Oct 15, 2025
2 checks passed
@Squidly271 Squidly271 deleted the Fix/VMSnapshot branch November 5, 2025 01:35
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.

3 participants